-
Notifications
You must be signed in to change notification settings - Fork 358
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 panda3d installation and example #1183
Fix panda3d installation and example #1183
Conversation
f85e6b6
to
842a254
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Panda3d viewer is broken.
@@ -121,3 +117,5 @@ def displayVisuals(self, visibility): | |||
"""Set whether to display visual objects or not.""" | |||
self.viewer.show_group(self.visual_group, visibility) | |||
self.display_visuals = visibility | |||
|
|||
__all__ = ['Panda3dVisualizer'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jcarpent removing Panda3dViewer
from __all__
breaks the examples written in Pinocchio.
I am not familiar with the code, but it seems we need to put back the previous __all___
definition.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cmastalli @ikalevatykh The changes are relevant, as panda3d_viewer may not be available. Then, you cannot import the lib, otherwise, you will get an error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jcarpent Original variant was also OK as panda3d_visualize
was not imported in visualize. _import_
. Just to clarify
The title is self-explanatory :)