search

enthought / mayavi

3D visualization of scientific data in Python
http://docs.enthought.com/mayavi/mayavi/
Other
1.28k stars 282 forks source link

Fix vtk 9.2.6 issues #1233

Closed prabhuramachandran closed 1 year ago

prabhuramachandran commented 1 year ago

This should fix recent build and test failures and also #1232.

prabhuramachandran commented 1 year ago

@homosapien-lcy, @mdickinson, @rahulporuri -- I've fixed what I can at this point. I am not sure why the pyside6 ubuntu Python 3.10 test is segfaulting, it doesn't seem to be anything to do with Mayavi/VTK. I will need to take a closer look at the headless test failure, the same test passes for me on Python 3.11 with VTK-9.2.6. Please review this PR and let me know how we should proceed. Thanks.

prabhuramachandran commented 1 year ago

Closing this as #1199 fixes these issues already.

Ataraxiainc1 commented 1 year ago

""

The error you are experiencing is a compilation error during the installation of the "mayavi" library. It appears to occur in the file "tvtk\src\array_ext.c" and is related to data type conversions.

The warning messages indicate possible data loss in the conversions from "size_t" to "int" or from "Py_ssize_t" to "int". These warnings can be ignored as they are not critical errors.

However, the actual error occurs on line "tvtk\src\array_ext.c(2313)" where you encounter an error "C2198: 'PyCode_New': too few arguments for call". This suggests that the function "PyCode_New" is being called with fewer arguments than required, resulting in a compilation error.

To fix this issue, you will need to review the source code of "mayavi" and ensure that the call to the "PyCode_New" function has the correct number of arguments. It may be necessary to update the "mayavi" library or contact the developers for further assistance with this specific problem.

Remember that the warning messages are just warnings and should not affect the overall functionality of the library once it is successfully compiled.

""