Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conditional 3D Display, useful to get around buggy Mayavi install #118

Open
wants to merge 7 commits into
base: force_cpu
Choose a base branch
from

Conversation

dccote
Copy link
Contributor

@dccote dccote commented Aug 3, 2024

An environment variable PYTISSUE_NO3DDISPLAY can be defined to bypass any Mayavi 3D display.

This appears necessary due to the difficulty to install Mayavi on all platforms: sometimes, traitsui fails (8.0.0 has broken things apparently, and forcing old version not always possible), sometimes vtk fails (must be installed via mayavi, which does not always recognize it), etc... By defining in your environment:

export  PYTISSUE_NO3DDISPLAY=1

you can avoid any buggy Mayavi install and run the code regardless, but without 3D visualization which is not required for the code to run.

An environment variable PYTISSUE_NO3DDISPLAY can be defined to bypass any Mayavi 3D display.  This appears necessary due to the difficulty to install Mayavi on all platforms: sometimes, traitsui fails, sometimes vtk fails, etc... By defining it in your environment, you can avoid bugs and run the code reagardless.
@dccote dccote self-assigned this Aug 3, 2024
@dccote dccote requested review from PyMarc2 and JLBegin August 3, 2024 22:37
@dccote dccote changed the title Conditional 3D Display Conditional 3D Display, useful to get around buggy Mayavi install Aug 3, 2024
@dccote dccote closed this Aug 5, 2024
@dccote dccote deleted the conditional_display branch August 5, 2024 16:26
@dccote dccote restored the conditional_display branch August 5, 2024 16:40
@dccote dccote reopened this Aug 5, 2024
@JLBegin
Copy link
Contributor

JLBegin commented Aug 11, 2024

I think your installation issues were related to mayavi being pinned to 4.8.1, which is not compatible with the latest vtk (default on fresh installs).

Considering the new mayavi release, removing the pin on traitsui with mayavi >= 4.8.2 should fix this.

@JLBegin
Copy link
Contributor

JLBegin commented Aug 11, 2024

I think you meant to target branch force_cpu.

@JLBegin JLBegin changed the base branch from main to force_cpu October 26, 2024 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

It would be very useful to be able to run the code even if mayavi and vtk are not installed
2 participants