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

Set armature for hinges #202

Open
surgura opened this issue Nov 30, 2022 · 3 comments · Fixed by #234
Open

Set armature for hinges #202

surgura opened this issue Nov 30, 2022 · 3 comments · Fixed by #234
Labels
bug Something isn't working as intended

Comments

@surgura
Copy link
Contributor

surgura commented Nov 30, 2022

Hinges currently have no armature set(motor has 0 intertia itself). This is actually not realistic and can cause unstable simulations and vibrating behaviour.

Also look at servo damping(internal friction). Not as important.
And mujoco's 'forcerange', which it does not pick up from the URDF file and we have to manually set. Also not that important.

@surgura surgura added this to the beta2 milestone Nov 30, 2022
@surgura
Copy link
Contributor Author

surgura commented Feb 2, 2023

@tckarenchiang We should pick this up as the next issue when you have time. This is important for students.

@oliverweissl
Copy link
Member

Adding armature helped a lot! This should definitely be added.

@surgura surgura added the bug Something isn't working as intended label May 25, 2023
@surgura surgura linked a pull request May 25, 2023 that will close this issue
@surgura surgura modified the milestones: beta2, Isaac Gym tracker May 25, 2023
@surgura
Copy link
Contributor Author

surgura commented May 25, 2023

Done for mujoco. Moving to isaac gym tracker. We will not solve it for isaac right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working as intended
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants