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.

Sign up for GitHub

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

Jump to bottom

ergocubSN000 has wrong names for l_upper_arm and r_upper_arm #197

Closed

traversaro opened this issue Dec 1, 2023 · 1 comment · Fixed by #198

Assignees

Labels

domain-software prj-ergocub team-fix

Member

traversaro commented Dec 1, 2023

As of 4a1ad6b, the wrong link names l_upperarm and r_upperarm are used instead (see https://icub-tech-iit.github.io/documentation/icub_kinematics/icub-model-naming-conventions/icub-model-naming-conventions/ for the reference on the convention for which they should be l_upper_arm, r_upper_arm.

The text was updated successfully, but these errors were encountered:

Nicogene self-assigned this

Nicogene added team-fix prj-ergocub domain-software labels

Nicogene added a commit that referenced this issue


          ergoCubV1*: correct names l/r_upperarm w/ r/l_upper_arm

ad0023d

It fixes #197

Nicogene mentioned this issue

ergoCubV1*: correct names l/r_upperarm w/ r/l_upper_arm #198

Merged

Member

Nicogene commented Dec 4, 2023

PR:

ergoCubV1*: correct names l/r_upperarm w/ r/l_upper_arm #198

Nicogene closed this as completed in #198

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment