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

Fix OpenGL not available crash #97

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

radekd91
Copy link

@radekd91 radekd91 commented Feb 2, 2024

When OpenGL cannot be properly imported (when it is not installed at all or when it doesn't work for some reason), the import of the library will crash with a NotImplementedError

This fix catches this error and allows the library to be imported without crashing. OpenGL related functionality (MeshViewer) will not work but at least the Mesh class can be used. Many users might only need that functionality.

PaulaMihalcea and others added 5 commits March 23, 2021 10:22
The current installation guide is unclear about the actual usage of the psbody-mesh package after building it, as well as impossible to execute on machines without sudo permissions.

The installation part of the documentation has been thus rewritten in order to make it more clear and complete, as well as sudo-free (by using Anaconda).

The parts with the download in the site-packages folder and the renaming of the mesh folder to psbody is what actually makes using this package possible, and this rectified version of the guide contains the cleanest possible way of doing it - the alternatives being copying the mesh folder after compilation in the folders of the projects that use it (bad) and/or editing the import lines in the relative scripts (even worse), as well as the PYTHONPATH variable.
Also updated steps 4 and 5 in the Conda installation section with fewer commands
This allows to use the Mesh class without OpenGL
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.

3 participants