Skip to content

sicks300_ros2: 1.4.0-1 in 'jazzy/distribution.yaml' [bloom] - #53795

Open
ajtudela wants to merge 1 commit into
ros:masterfrom
ajtudela:bloom-sicks300_ros2-1
Open

sicks300_ros2: 1.4.0-1 in 'jazzy/distribution.yaml' [bloom]#53795
ajtudela wants to merge 1 commit into
ros:masterfrom
ajtudela:bloom-sicks300_ros2-1

Conversation

@ajtudela

@ajtudela ajtudela commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Increasing version of package(s) in repository sicks300_ros2 to 1.4.0-1:

sicks300_ros2

* Add a unit test suite for the parser, scanner and scan filter.
* Move serial acquisition to a dedicated thread behind an injectable ``ISerialIO`` transport interface.
* Give the serial read a real wall-clock timeout and make the communication timeout report an error.
* Consolidate diagnostics behind ``diagnostic_updater``.
* Guard against degenerate scans and keep ``angle_min``/``angle_max``, ``angle_increment`` and ``scan_time`` consistent.
* Fix the receive buffer compaction in ``getScan``.
* Replace unaligned ``reinterpret_cast`` reads in the parser with ``memcpy``.
* Remove static state shared between scanner instances.
* Remove the inert scan timestamp synchronization, dead members, stubs and commented-out code.
* Extract ``declare_parameter_if_not_declared`` to a shared header.
* Rename Hungarian-notation identifiers to ``snake_case`` and name magic numbers.
* Pass vectors and strings by const reference.
* Deprecate the ``scan_filter`` node in favor of the ``laser_filters`` package and remove the old laser filter.
* Declare ``lower_angle_``/``upper_angle_`` as ``double`` in ``scan_filter``.
* Rename package to ``sicks300_ros2`` and update publisher QoS.
* Update the CI workflow.

@github-actions github-actions Bot added the jazzy Issue/PR is for the ROS 2 Jazzy distribution label Sep 7, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

For changes related to rosdistro:

  • ✅ New packages are named appropriately
  • ✅ An up-to-date Bloom version was used
  • ✅ Release changes are confined to a single distribution

For changes related to yamllint:

  • ✅ All new lines of YAML pass linter checks

@ahcorde ahcorde left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@ahcorde ahcorde added the held for sync Issue/PR has been held because the distribution is in a sync hold label Sep 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

held for sync Issue/PR has been held because the distribution is in a sync hold jazzy Issue/PR is for the ROS 2 Jazzy distribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants