tier4 / nebula

A universal LiDAR and radar driver for ROS 2, supporting Hesai, Velodyne, Robosense and Continental sensors.
https://tier4.github.io/nebula/
Apache License 2.0
51 stars 51 forks source link

feat: pretty-print sensor configurations #172

Closed mojomex closed 4 months ago

mojomex commented 4 months ago

PR Type

Description

Sensor configurations were printed without any special formatting before, making it hard to quickly, at a glance, verify sensor settings. This PR proposes printing configs in a prettier format. See the below table for comparisons.

Before After
Screenshot from 2024-06-27 16-57-20 Screenshot from 2024-06-27 16-48-51
Screenshot from 2024-06-27 16-49-26
Screenshot from 2024-06-27 16-57-34 Screenshot from 2024-06-27 16-49-51
Screenshot from 2024-06-27 16-50-16
Screenshot from 2024-06-27 16-50-38

Additionally,

Pre-Review Checklist for the PR Author

PR Author should check the checkboxes below when creating the PR.

Checklist for the PR Reviewer

Reviewers should check the checkboxes below before approval.

Post-Review Checklist for the PR Author

PR Author should check the checkboxes below before merging.

CI Checks