ami-iit / bipedal-locomotion-framework

Suite of libraries for achieving bipedal locomotion on humanoid robots
https://ami-iit.github.io/bipedal-locomotion-framework/
BSD 3-Clause "New" or "Revised" License
132 stars 36 forks source link

RT Capabilities Part 1 - YARP Logger Device #817

Closed nicktrem closed 3 months ago

nicktrem commented 4 months ago

The first part of the real-time capabilities functionality for the YARP logger device. This PR is a smaller portion of this original pull request for the logger device and just contains the bare-bone software required to get real-time logging up and running. The original pull request for the logger device will be broken up into this pull request and others to simplify the merge process

traversaro commented 4 months ago

The macOS failure is:

2024-03-11T09:02:15.9093100Z Error: The `brew link` step did not complete successfully
2024-03-11T09:02:15.9095690Z The formula built, but is not symlinked into /usr/local
2024-03-11T09:02:15.9099740Z Could not symlink lib/node_modules/npm/node_modules/@npmcli/disparity-colors/node_modules/ansi-styles/index.js
2024-03-11T09:02:15.9103080Z Target /usr/local/lib/node_modules/npm/node_modules/@npmcli/disparity-colors/node_modules/ansi-styles/index.js
2024-03-11T09:02:15.9105240Z already exists. You may want to remove it:
2024-03-11T09:02:15.9107510Z   rm '/usr/local/lib/node_modules/npm/node_modules/@npmcli/disparity-colors/node_modules/ansi-styles/index.js'
2024-03-11T09:02:15.9109480Z 
2024-03-11T09:02:15.9110010Z To force the link and overwrite all conflicting files:
2024-03-11T09:02:15.9111330Z   brew link --overwrite node@18
2024-03-11T09:02:15.9111960Z 
2024-03-11T09:02:15.9112310Z To list all files that would be deleted:
2024-03-11T09:02:15.9113650Z   brew link --overwrite node@18 --dry-run
2024-03-11T09:02:15.9114480Z 

it is unrelated to the PR, and kind the npm equivalent of https://github.com/actions/setup-python/issues/577 .

@nicktrem You can safely ignore this failure.

@GiulioRomualdi @S-Dafarra For me we can safely disable homebrew CI.

GiulioRomualdi commented 3 months ago

Hi @nicktrem! Thanks for iterating! I'm going to merge the PR well done!

PS; sorry for the delay 😭

DanielePucci commented 3 months ago

Great work all!