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

Bump human-dynamics-estimation version to 3.0.0 #375

Merged
merged 4 commits into from
Nov 14, 2023
Merged

Conversation

lrapetti
Copy link
Member

@lrapetti lrapetti commented Nov 9, 2023

Closes #373.
Before merging:

Changed

Previous devices name New devices name
HumanDynamicsWrapper HumanDynamics_nws_yarp
HumanDynamicsRemapper HumanDynamics_nwc_yarp
HumanStateWrapper HumanState_nws_yarp
HumanStateRemapper HumanState_nwc_yarp
HumanWrenchWrapper HumanWrench_nws_yarp
HumanWrenchRemapper HumanWrench_nwc_yarp
WearableTargetsWrapper WearableTargets_nws_yarp
WearableTargetsRemapper WearableTargets_nwc_yarp

Added

@lrapetti lrapetti self-assigned this Nov 9, 2023
Copy link
Contributor

@davidegorbani davidegorbani left a comment

Choose a reason for hiding this comment

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

It looks fine to me!

@S-Dafarra
Copy link
Contributor

S-Dafarra commented Nov 9, 2023

Could you wait for this? would like to propose changes for using the new frameTransformClient device

I am planning to do this either today or tomorrow

@lrapetti
Copy link
Member Author

lrapetti commented Nov 9, 2023

Could you wait for this? would like to propose changes for using the new frameTransformClient device

Sure! As discussed today, before merging we would also like to improve the documentation on the required changes to use the new nws/nwc devices. I will update the first comment of this issue

@davidegorbani
Copy link
Contributor

I updated the CHANGELOG file as suggested by @traversaro .

@S-Dafarra
Copy link
Contributor

Could you wait for this? would like to propose changes for using the new frameTransformClient device

I am planning to do this either today or tomorrow

Here the PR: #377

@lrapetti
Copy link
Member Author

Thanks all, we can merge it

@lrapetti lrapetti merged commit b5c6e44 into master Nov 14, 2023
6 checks passed
@lrapetti lrapetti deleted the release-v3.0.0 branch November 14, 2023 12:47
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.

Release human-dynamics-estimation v3.0.0
4 participants