-
Notifications
You must be signed in to change notification settings - Fork 761
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
Tendons in USD #1714
Tendons in USD #1714
Conversation
abhihjoshi
commented
Jun 3, 2024
- Adding support for tendons in USD
- Updating geom visibility logic for changing visibility across multiple frames
- Adding user friendly demo script for generating USD file for inputted XML file
Demo script
Shape meshes
adding demo file and unnamed geom fix
…ding frame skip arg to demo
bc27690
to
ea128c1
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.
Hi, I used the branch and the exported file name will have an additional _
. Is this intentional or a typo?
python/mujoco/usd/exporter.py
Outdated
# post-processing for visibility of geoms in scene | ||
for _, geom_ref in self.geom_refs.items(): | ||
geom_ref.update_visibility(False, geom_ref.last_visible_frame+1) | ||
|
||
self.stage.Export( | ||
f"{self.output_directory_root}/{self.output_directory_name}/frames/frame_{self.frame_count}_.{filetype}" |
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.
Hi, is this a typo to have an additional _
after {self.frame_count}
?
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.
I remember this was intentional prior to when we were able to render a full trajectory. However, now, I do not believe there is a reason. I can remove this.
60afda9
into
google-deepmind:main