Skip to content

Conversation

@ahcorde
Copy link
Contributor

@ahcorde ahcorde commented Nov 6, 2025

Related with #1616

@nbbrooks do you mind to take a look ?

Signed-off-by: Alejandro Hernandez Cordero <[email protected]>
Copy link
Contributor

@nbbrooks nbbrooks left a comment

Choose a reason for hiding this comment

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

I can confirm these changes work!

  1. build rviz2 from source with the colcon build --merge-install option
  2. Add a default Pointcloud2 widget and see the policy now defaults to Best Effort
  3. Publish an example pointcloud using best effort profile and verify points are visible
  4. Manually switch widget policy to Reliable to repro old behavior
ros2 topic pub /test_cloud sensor_msgs/msg/PointCloud2 "{header: {frame_id: map}, height: 1, width: 3, fields: [{name: x, offset: 0, datatype: 7, count: 1}, {name: y, offset: 4, datatype: 7, count: 1}, {name: z, offset: 8, datatype: 7, count: 1}], is_bigendian: false, point_step: 12, row_step: 36, data: [0,0,0,0,0,0,0,0,0,0,0,0, 0,0,128,63,0,0,0,0,0,0,0,0, 0,0,128,63,0,0,128,63,0,0,0,0], is_dense: true}" --qos-reliability best_effort
simplescreenrecorder-2025-12-06_13.01.27.mp4

@ahcorde
Copy link
Contributor Author

ahcorde commented Dec 8, 2025

Pulls: #1621
Gist: https://gist.githubusercontent.com/ahcorde/982b1df8a6f90eeea6688e743b3d83c7/raw/97603b2b5d66ec7e4cc7f244540e09637cd09b6e/ros2.repos
BUILD args: --packages-above-and-dependencies rviz_common rviz_default_plugins
TEST args: --packages-above rviz_common rviz_default_plugins
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/17710

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@ahcorde ahcorde merged commit 6747294 into rolling Dec 8, 2025
2 checks passed
@ahcorde ahcorde deleted the ahcorde/rolling/pointcloud2_best_effort branch December 8, 2025 17:04
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