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

SW-5167: follow ups from SW-5167-fix-black-columns-when-recording-under-ros-foxy #163

Merged
merged 2 commits into from
Jul 14, 2023

Conversation

Samahu
Copy link
Contributor

@Samahu Samahu commented Jul 10, 2023

Related Issues & PRs

Summary of Changes

  • Initialize point_cloud_frame with an empty string
  • Expose cloud_frame through launch xml files
  • Remove unused/duplicate param definitions
  • Update launch files args descriptions

Validation

Launch a live sensor using the xml launch file format setting only the value of the sensor_hostname
ros2 launch ouster_ros sensor.launch.xml
sensor_hostname:=
Once the sensor connects verify that all topics are using a BEST_EFFORT policy using:

ros2 topic echo -v "topic_name"
Launch record.launch.xml as follows:
ros2 launch ouster_ros record.launch.xml
sensor_hostname:=
bag_file:=
Once the sensor connects verify that all topics are using a RELIABLE policy using:

ros2 topic echo -v "topic_name"

…cloud_frame through launch xml files +

Remove unused/duplicate param definitions +
Update descriptions
@Samahu Samahu added the enhancement New feature or request label Jul 10, 2023
@Samahu Samahu self-assigned this Jul 10, 2023
@Samahu Samahu marked this pull request as ready for review July 11, 2023 23:25
Copy link
Contributor

@kairenw kairenw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. I didn't run it - let me know if you want another test

@Samahu Samahu merged commit 9cbb840 into ros2 Jul 14, 2023
3 checks passed
@Samahu Samahu deleted the SW-5167-follow-ups-for-ros2-branch branch July 14, 2023 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants