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.

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

Add VectorCollectionServer to dump the desired/feedback joint/axis values #144

Merged
merged 11 commits into from
Sep 13, 2024
Merged

Add VectorCollectionServer to dump the desired/feedback joint/axis values #144

merged 11 commits into from
Sep 13, 2024

Conversation

S-Dafarra
Copy link
Collaborator

@S-Dafarra S-Dafarra commented Jul 30, 2024 •
edited
Loading

Fixes #110

@S-Dafarra S-Dafarra force-pushed the stream_desired_values branch from 27c4f83 to 72f3ba5 Compare July 30, 2024 09:32
@S-Dafarra
Copy link
Collaborator Author

With 72f3ba5 I have changed the CI installation from vcpkg/source to conda. On the other other hand, I just released wearables (https://github.com/robotology/wearables/releases/tag/v1.9.0) and it is not available yet in conda. Hence, for the time being, the HapticGloveModule compilation is not tested in CI.

S-Dafarra added 11 commits August 1, 2024 11:26
@S-Dafarra
Updated gitignore to ignore vs files
ed28d72
@S-Dafarra
Use conda in CI instead of vcpkg and source installation
34039a4
@S-Dafarra
Removed files that are included by YCM too
7ef4f25
@S-Dafarra
Removed matlogger2 dependency and removed logging from OculusModule
39ec446 
It is not used anymore anyhow
@S-Dafarra
Initialize the m_n variable in RobotMotorsEstimation
3333130 
See #145
@S-Dafarra
Using bipedal as optional dependency
717f39d
@S-Dafarra
Return directly the vector of names for human and controlled robot jo…
7db1427 
…ints

This simplifies the logger preparation
@S-Dafarra
Added possibility to get fingersTactileData
6f6b5b9
@S-Dafarra
Added possibility to retrieve more data from the teleoperation object
2e6d978
@S-Dafarra
Using bipedal vectors collection server to log data in HapticGloveModule
879a7dc
@S-Dafarra
Updated haptic glove configuration files with new logger options
18606f6
@S-Dafarra S-Dafarra force-pushed the stream_desired_values branch from 3d54ff5 to 18606f6 Compare August 1, 2024 10:19
@S-Dafarra
Copy link
Collaborator Author

I have finished the process of removing the old logger in favor of blf vectors collection. I have also cleaned the history for a more clear separation of changes between the commits.

@S-Dafarra
Copy link
Collaborator Author

It needs to be tested on a robot

@S-Dafarra S-Dafarra mentioned this pull request Aug 1, 2024
Merged
@S-Dafarra S-Dafarra marked this pull request as ready for review August 9, 2024 12:33
@S-Dafarra
Copy link
Collaborator Author

Tested on the robot. Ready for review.

@S-Dafarra
Copy link
Collaborator Author

With 72f3ba5 I have changed the CI installation from vcpkg/source to conda. On the other other hand, I just released wearables (robotology/wearables@v1.9.0 (release)) and it is not available yet in conda. Hence, for the time being, the HapticGloveModule compilation is not tested in CI.

This still seems to be the case, but probably it is blocked by robotology/robotology-superbuild#1678

@traversaro
Copy link
Member

With 72f3ba5 I have changed the CI installation from vcpkg/source to conda. On the other other hand, I just released wearables (robotology/wearables@v1.9.0 (release)) and it is not available yet in conda. Hence, for the time being, the HapticGloveModule compilation is not tested in CI.

This still seems to be the case, but probably it is blocked by robotology/robotology-superbuild#1678

It is blocked by that and fixing the conda packages generation. I can look into that in the weekend.

@S-Dafarra S-Dafarra closed this Sep 9, 2024
@S-Dafarra S-Dafarra reopened this Sep 9, 2024
@S-Dafarra
Copy link
Collaborator Author

After robotology/robotology-superbuild#1670 (comment), wearables version have been updated in conda. Triggered again the CI

@S-Dafarra
Copy link
Collaborator Author

CI is happy. @GiulioRomualdi let me know if you have any comment

@S-Dafarra S-Dafarra closed this Sep 10, 2024
@S-Dafarra S-Dafarra reopened this Sep 10, 2024
@S-Dafarra S-Dafarra merged commit 95ed1a6 into master Sep 13, 2024
12 checks passed
@S-Dafarra S-Dafarra deleted the stream_desired_values branch September 13, 2024 15:23
This was referenced Sep 13, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GiulioRomualdi GiulioRomualdi GiulioRomualdi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Switch to BLF logger from MatLog2
3 participants
@S-Dafarra @traversaro @GiulioRomualdi