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

Mismatch bewteen the ankle roll motor model of robots configuration and the documentation. #360

Closed
GiulioRomualdi opened this issue Sep 12, 2024 · 6 comments · Fixed by robotology/robots-configuration#678
Assignees

Comments

@GiulioRomualdi
Copy link
Contributor

I was checking the motors model mounted on ergoCubSN001 and the one in the documentation https://icub-tech-iit.github.io/documentation/ergoCub1/ergoCub1-joints/

Mismatch Joint Motor in the documentation Motor in robots-config Link of robots-config
🟢 shoulder pitch MOOG_C2900576 MOOG-BL-C2900576 here
🟢 shoulder roll MOOG_C2900576 MOOG-BL-C2900576 here
🟢 shoulder yaw MOOG_C2900575 MOOG-BL-C2900575 here
🟢 elbow MOOG_C2900575 MOOG-BL-C2900575 here
🟢 torso roll MOOG_C2900576 MOOG_C2900576 here
🟢 torso_pitch MOOG_C2900576 MOOG_C2900576 here
🟢 torso yaw MOOG_C2900576 MOOG_C2900576 here
🟢 hip pitch MOOG_C2900580 MOOG_C2900580 here
🟢 hip roll MOOG_C2900576 MOOG_C2900576 here
🟢 hip yaw MOOG_C2900576 MOOG_C2900576 here
🟢 knee MOOG_C2900580 MOOG_C2900580 here
🟢 ankle pitch MOOG_C2900580 MOOG_C2900580 here
ankle roll MOOG_C2900576 MOOG_C2900575 here

Which is the correct model mounted on the robot?

@GiulioRomualdi
Copy link
Contributor Author

@pattacini pattacini self-assigned this Sep 12, 2024
@pattacini
Copy link
Member

cc @fiorisi

@fiorisi
Copy link
Member

fiorisi commented Sep 12, 2024

@GiulioRomualdi in the robot snapshot we have the MOOG_C2900576.

image

@pattacini
Copy link
Member

I'll update the robots conf files accordingly then.

@pattacini
Copy link
Member

I'll update the robots conf files accordingly then.

See robotology/robots-configuration#678.

I've applied the same fix to all the ergoCub robots, blindly. If this is not correct, I'll change the PR. Let me know @fiorisi

@pattacini
Copy link
Member

I've applied the same fix to all the ergoCub robots, blindly. If this is not correct, I'll change the PR. Let me know @fiorisi

Verified with @fiorisi: all ergoCub robots mount the same ankle motor.

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 a pull request may close this issue.

3 participants