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.

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

Ros2 launch #25

Merged
merged 2 commits into from
Jun 14, 2023
Merged

Ros2 launch #25

merged 2 commits into from
Jun 14, 2023

Conversation

JosefGst
Copy link

@JosefGst JosefGst commented May 8, 2023

Im using the wit imu in ros2 humbel and the launch file could only be recognized if I renamed them to end with _launch or .launch

JosefGst and others added 2 commits May 8, 2023 12:01
launch files need to end with _launch or .launch to be install by colcon.
remove additional creation of another second src folder
@twdragon
Copy link
Collaborator

twdragon commented May 8, 2023

@fllay can you please test the changes on ROS2 stack? Then I could mention you as a reviewer

@fllay
Copy link
Contributor

fllay commented May 8, 2023 via email

@twdragon
Copy link
Collaborator

twdragon commented May 9, 2023

I will test it this weekend

On Mon, 8 May 2023 at 20:01 Andrei Vukolov @.> wrote: @fllay https://github.com/fllay can you please test the changes on ROS2 stack? Then I could mention you as a reviewer — Reply to this email directly, view it on GitHub <#25 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA6HLGYOUKHHNOQSMIBZJ6DXFDVCZANCNFSM6AAAAAAXZNU2JA . You are receiving this because you were mentioned.Message ID: @.>

@fllay thank you! Looking forward to seeing the test report from you!

@fllay
Copy link
Contributor

fllay commented May 16, 2023

I tested with WT61C. It seems to be OK.

@twdragon
Copy link
Collaborator

@fllay thanks, so I consider this PR to merge.

@gsokoll gsokoll mentioned this pull request May 21, 2023
@twdragon twdragon merged commit 7252d44 into ElettraSciComp:ros2 Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants