mlab.test_imshow() fail about mayavi HOT 5 CLOSED

The original code causing the problem is here (mayavi/tools/modules.py:72):

def _line_width_changed(self):
    try:
        self._target.actor.property.line_width = self.line_width
    except AttributeError:
        try:
            self._target.property.line_width = self.line_width
        except AttributeError:
            pass

It turns out that when "line_width" is not an attribute of whatever object is actor.property or _target.property, a TraitError is thrown (for at least some items) and not an AttributeError, which is what would be handled by the exception handlers above. TraitError is defined in another package called traits, which is in a separate folder to mayavi. I have modified my local copy of modules.py:72 to look like this:

def _line_width_changed(self):
    if hasattr(self._target, 'actor') and \
       hasattr(self._target.actor, 'property') and \
       hasattr(self._target.actor.property, 'line_width'):
        self._target.actor.property.line_width = self.line_width
    elif hasattr(self._target, 'property') and \
       hasattr(self._target.property, 'line_width'):
        self._target.property.line_width = self.line_width

Basically, I am explicitly testing for the presence of the desired attributes. I have nowhere near enough knowledge about the mayavi codebase to know whether this is the right way to approach this problem or not. Adding an import (from traits.api import TraitError) and adding TraitError to the exception handlers in the original code feels icky to me.

from mayavi.

I'm having the same problem.

from mayavi.

A also have this problem, my environment Windows 7, python(x,y) distribution.

Thank you for your patch, it solve this problem in my environment!

from mayavi.

Same here on Mac OS X 10.8.4, with MacPorts Python 2.7.6, py27-mayavi @4.2.0_1. The proposed patch fixes the error, in my case triggered when running the pick_on_surface.py example.

from mayavi.

Fixed with #96.

from mayavi.