-
Notifications
You must be signed in to change notification settings - Fork 27
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
Joint positions might incorrect after RESET #33
Comments
afaik this was done because the motors in the simulation do not support multi-rotation for position controls, so in normal operation the command values need to be between -pi and pi |
The simulation on the develop branch handles the joint position correctly, even for multi-turn motors. I also added a develop branch for the orogen mars component where the 2pi conversion is removed. It would be great if you can test the develop branch of mars and orogen/mars to check if the simulation behaves correct for you? |
I finally had time to test this feature on the develop branch, seems ok to me |
Can you merge this feature to master? |
It should already be in the master branch since 10th of January ;-) |
Ahh, I see, I only checked the branch of MARS not of the orogen component ;-) |
Merged :-) |
One more (:
When I do a RESET, the joint positions might be flipped by 2pi. The problem occurs when I reset the simulation while one or more of the joints are located at position >pi or <-pi.
This is due to the absolute position conversion done here:
simulation-orogen-mars/tasks/Joints.hpp
Line 43 in a6ea2a4
Is there a good reason for doing such a conversion? If not I would suggest to remove it.
The text was updated successfully, but these errors were encountered: