search

robotology / walking-teleoperation

Software related to walking and teleoperation.
BSD 3-Clause "New" or "Revised" License
30 stars 14 forks source link

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

Closed S-Dafarra closed 1 month ago

S-Dafarra commented 2 months ago

Fixes https://github.com/robotology/walking-teleoperation/issues/110

S-Dafarra commented 2 months ago

With https://github.com/robotology/walking-teleoperation/pull/144/commits/72f3ba5fe6299668b232afbd420c9227819ecd25 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 commented 2 months ago

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 commented 2 months ago

It needs to be tested on a robot

S-Dafarra commented 2 months ago

Tested on the robot. Ready for review.

S-Dafarra commented 2 months ago

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 https://github.com/robotology/robotology-superbuild/pull/1678

traversaro commented 2 months ago

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 commented 1 month ago

After https://github.com/robotology/robotology-superbuild/issues/1670#issuecomment-2334052798, wearables version have been updated in conda. Triggered again the CI

S-Dafarra commented 1 month ago

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