-
Notifications
You must be signed in to change notification settings - Fork 8
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
Added mocap interfaces to comply with Rep mocap #21
base: rep-mocap
Are you sure you want to change the base?
Conversation
Signed-off-by: Alberto García <[email protected]>
Signed-off-by: Alberto García <[email protected]>
Signed-off-by: Alberto García <[email protected]>
Signed-off-by: Alberto García <[email protected]>
Signed-off-by: Alberto García <[email protected]>
Signed-off-by: Alberto García <[email protected]>
Signed-off-by: Alberto García <[email protected]>
Signed-off-by: Alberto García <[email protected]>
Hello @MFernandezCarmona, We have modified the driver to align with the proposed REP standard under discussion (referenced here). Could you please test it on the actual Vicon system and report any issues or inconsistencies encountered? Thank you in advance. |
Signed-off-by: Alberto García <[email protected]>
Update the repositories leaving only mocap4ros2_vicon
Hi! Thanks for your work! |
Thank you very much for your quick reply. No problem we will wait until the mocap is available. Best regards! |
Signed-off-by: Alberto García <[email protected]>
I hope this message finds you well. Best regards, |
Hi, |
Thank you! We will wait until the motion capture system is available. We were looking to test the changes introduced, which essentially replace mocap_msgs with mocap_interfaces to align with the REP. Best, |
Modify mocap4r2_msgs to mocap_interfaces to comply with Rep, style modified to comply with cpplint and uncrustify