robotology / wearables

Code moved to https://github.com/robotology/human-dynamics-estimation
https://github.com/robotology/human-dynamics-estimation
BSD 3-Clause "New" or "Revised" License
19 stars 11 forks source link

Update IWear Logger with Yarp port streaming #123

Closed yeshasvitirupachuri closed 3 years ago

yeshasvitirupachuri commented 3 years ago

This PR updates iwear_logger device with optional streaming of data to yarp ports. This has been tested using iFeel node data

https://user-images.githubusercontent.com/6505998/122778008-e0a43d80-d2ac-11eb-836f-5f6b069ad791.mp4

CC @prashanthr05 @lrapetti @claudia-lat @DanielePucci

yeshasvitirupachuri commented 3 years ago

Tests following PR Changes https://github.com/robotology/wearables/pull/123/commits/afacfdf1c362f7bf87ef28f853f25037a6a3ca8a

https://user-images.githubusercontent.com/6505998/123400187-c7013f80-d5a5-11eb-8768-7af9c468c945.mp4

lrapetti commented 3 years ago

With https://github.com/robotology/wearables/pull/123/commits/64ef5fcadf2b5ca563c0e232cc90c8cfa2ea848b the compilation failure in MacOS should be solved.

We can wait for the CI to confirm it, and then if there are not further comments we can merge the PR. @Yeshasvitvs please check if you want to clean the commit history or we can squash the commits.

yeshasvitirupachuri commented 3 years ago

We can wait for the CI to confirm it, and then if there are not further comments we can merge the PR.

@lrapetti Thanks, the CI is fixed with the change. However, this I was based on commit that added YARP::YARP_os for testing the CI.

please check if you want to clean the commit history or we can squash the commits.

I dropped those commits, and added YARP:YARP_init again in https://github.com/robotology/wearables/pull/123/commits/07d924e6a02abafd6c8601ff8200812301e36e08. Sorry about the mess 😅

Following this you can squash the comments and merge. Thaks.