Bug 620068

vtk is now in the buildroot; I'm looking at Mayavi

Created attachment 435926 [details]
Fix for UnboundLocalError in wrapper_gen.py on py2.7

Fix for this error:

  File "enthought/tvtk/wrapper_gen.py", line 456, in _gen_get_set_methods
UnboundLocalError: local variable 'rng' referenced before assignment

With the above patch, build gets further, but fails with:
running build
----------------------------------------------------------------------
Building TVTK classes... SyntaxError: ('EOL while scanning string literal', ('tvtk_classes/delimited_text_reader.py', 105, 45, '    utf8_record_delimiters = traits.String(r"\n'))
error: tvtk_classes/delimited_text_reader.pyc: No such file or directory

Seems to be Mayavi-3.3.0/enthought/tvtk/setup.py in "gen_tvtk_classes_zip", which generates python source code into tvtk_classes.zip.

Unfortunately the error means that the zipfile is never actually written to the filesystem (I think), so debugging this is tricky.

Looks like mismatching " vs ' characters.

Hacking Mayavi-3.3.0/enthought/tvtk/wrapper_gen.py:WrapperGenerator.generate_code
and adding:
   out = sys.stdout
allows us to see the generated code.

I misread the error in comment #3, the generated code looks like this:

    utf8_record_delimiters = traits.String(r"
    ", enter_set=True, auto_set=False, help=\
        """
        Specify the character(s) that will be used to separate records.
        The order of characters in the string does not matter.  Defaults
        to "\r\n".
        """
    )
 
where clearly the newline within the r"" string literal is a syntax error.

Setting
  out = sys.stdout 
means we don't actually get the code written out; build fails later on IIRC.

Better debugging method: I added this to the top of generate_code

        class Tee:
            def __init__(self, out):
                self.out = out
            def write(self, data):
                self.out.write(data)
                sys.stdout.write(data)
	out = Tee(out)

to see the code on stdout _and_ to have it written out.

I attempted a fix for this by changing the code generator to emit multiline string literals, which got things further, now fails here:

SyntaxError: ('invalid syntax', ('tvtk_classes/renderer.py', 222, 10, '    pass = traits.Property(_get_pass, _set_pass, help=\\\n'))

The generated code is:
    pass = traits.Property(_get_pass, _set_pass, help=\
        """
        Set/Get a custom render pass.
        Initial value is NULL.
        """
    )

Given that the error occurs in column 10, the issue seems to be that "pass" is a reserved word.

Did this code ever get built for python 2.6?

Hi Dava,


Mayavi has a very active upstream, would mind to report this issue to upstream?

https://svn.enthought.com/enthought/report
http://code.enthought.com/support/

FYI, a bugfix version 3.3.2 is available at http://www.enthought.com/repo/ETS/ for several months, I haven't tried if it can work with python 2.7.

Mayavi 3.3.2 built sucessfully with python 2.7. 

See http://koji.fedoraproject.org/koji/taskinfo?taskID=2391288

Mayavi-3.3.2-1.fc14,python-Traits-3.4.0-1.fc14,python-TraitsGUI-3.4.0-2.fc14,python-TraitsBackendQt-3.4.0-1.fc14,python-TraitsBackendWX-3.4.0-3.fc14,python-EnvisagePlugins-3.1.2-1.fc14,python-EnvisageCore-3.1.2-1.fc14,python-EnthoughtBase-3.0.5-1.fc14,python-AppTools-3.3.2-1.fc14 has been submitted as an update for Fedora 14.
http://admin.fedoraproject.org/updates/Mayavi-3.3.2-1.fc14,python-Traits-3.4.0-1.fc14,python-TraitsGUI-3.4.0-2.fc14,python-TraitsBackendQt-3.4.0-1.fc14,python-TraitsBackendWX-3.4.0-3.fc14,python-EnvisagePlugins-3.1.2-1.fc14,python-EnvisageCore-3.1.2-1.fc14,python-EnthoughtBase-3.0.5-1.fc14,python-AppTools-3.3.2-1.fc14

Mayavi-3.3.2-1.fc14, python-Traits-3.4.0-1.fc14, python-TraitsGUI-3.4.0-2.fc14, python-TraitsBackendQt-3.4.0-1.fc14, python-TraitsBackendWX-3.4.0-3.fc14, python-EnvisagePlugins-3.1.2-1.fc14, python-EnvisageCore-3.1.2-1.fc14, python-EnthoughtBase-3.0.5-1.fc14, python-AppTools-3.3.2-1.fc14 has been pushed to the Fedora 14 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update Mayavi python-Traits python-TraitsGUI python-TraitsBackendQt python-TraitsBackendWX python-EnvisagePlugins python-EnvisageCore python-EnthoughtBase python-AppTools'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/Mayavi-3.3.2-1.fc14,python-Traits-3.4.0-1.fc14,python-TraitsGUI-3.4.0-2.fc14,python-TraitsBackendQt-3.4.0-1.fc14,python-TraitsBackendWX-3.4.0-3.fc14,python-EnvisagePlugins-3.1.2-1.fc14,python-EnvisageCore-3.1.2-1.fc14,python-EnthoughtBase-3.0.5-1.fc14,python-AppTools-3.3.2-1.fc14

Mayavi-3.3.2-1.fc14, python-Traits-3.4.0-1.fc14, python-TraitsGUI-3.4.0-2.fc14, python-TraitsBackendQt-3.4.0-1.fc14, python-TraitsBackendWX-3.4.0-3.fc14, python-EnvisagePlugins-3.1.2-1.fc14, python-EnvisageCore-3.1.2-1.fc14, python-EnthoughtBase-3.0.5-1.fc14, python-AppTools-3.3.2-1.fc14 has been pushed to the Fedora 14 stable repository.  If problems still persist, please make note of it in this bug report.