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

Devel ergocub sn002 #692

Merged
merged 19 commits into from
Dec 6, 2024
Merged

Conversation

pattacini
Copy link
Member

Alignment in view of distro 2024.11.0.

@pattacini pattacini self-assigned this Nov 28, 2024
@pattacini
Copy link
Member Author

cc @Nicogene

Copy link
Member Author

@pattacini pattacini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some FT are disabled.
cc @S-Dafarra @GiulioRomualdi

ergoCubSN002/calibrators/left_arm-calib.xml Outdated Show resolved Hide resolved
ergoCubSN002/calibrators/right_arm-calib.xml Outdated Show resolved Hide resolved
@S-Dafarra
Copy link
Contributor

The robot just went into maintenance to fix the wrists, the neck, and (in theory) also the broken temperature sensor on the ankle

@pattacini
Copy link
Member Author

The robot just went into maintenance to fix the wrists, the neck, and (in theory) also the broken temperature sensor on the ankle

Same as #691 (comment).

@traversaro
Copy link
Member

Sorry, I accidentally solved the conflict of the PR, not sure if this is done according the workflow. Anyhow, today with @S-Dafarra we were able to run the robot, and even if we only did the teleoperation test with a mixed YARP 3.10/YARP 3.9 setup. So I think at this point we can merge this and release 2024.11.0, what do you think @Nicogene @pattacini ?

@pattacini
Copy link
Member Author

Great! I'll merge the PR and tag the repo.

@pattacini
Copy link
Member Author

Apparentely, I cannot rebase & merge1 because of

image

Footnotes

  1. We ought to rebase & merge not to lose intermediate commits but to keep history a bit cleaner.

@pattacini
Copy link
Member Author

Hi @traversaro

Is this fork/branch already rebased on top of upstream devel?

@traversaro
Copy link
Member

Hi @traversaro

Is this fork/branch already rebased on top of upstream devel?

I am afraid not, I accidentally did a conflict resolution from the GitHub UI (that we can drop), but I did not interacted with the branch in any other way.

@pattacini
Copy link
Member Author

pattacini commented Dec 6, 2024

It's not rebased. Although the conflicts are solved, GitHub is unable to rebase & merge via the GUI.
I'll try to rebase it.

@pattacini pattacini merged commit d164d4c into robotology:devel Dec 6, 2024
1 check passed
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 this pull request may close these issues.

6 participants