Skip to content

added lifecycle node support, handling of acquisition erros#254

Merged
berndpfrommer merged 2 commits into
humble-develfrom
support_lifecycle_node
Oct 30, 2025
Merged

added lifecycle node support, handling of acquisition erros#254
berndpfrommer merged 2 commits into
humble-develfrom
support_lifecycle_node

Conversation

@berndpfrommer

Copy link
Copy Markdown
Collaborator

This PR

  • adds support for lifecycle nodes for distros where the image transport supports this
    (version must be >= 6.4.0). At the moment, this is only ROS Rolling.
  • improves handling of camera acquisition errors. The driver will try to restart acquisition automatically. However, if the camera lost power and rebooted, the driver will not automatically re-apply any of the camera parameter settings.

@jclinton830

Copy link
Copy Markdown

Tested with rolling, and it seems to be working well. Thanks for making the changes. @berndpfrommer

@stijn-vanesch

stijn-vanesch commented Oct 25, 2025

Copy link
Copy Markdown

You mentioned it is only working on Rolling. What is still open or would it take to have this working on Jazzy? Or has it simple not been tested on this Distro?

@berndpfrommer

Copy link
Copy Markdown
Collaborator Author

You mentioned it is only working on Rolling. What is still open or would it take to have this working on Jazzy? Or has it simple not been tested on this Distro?

The driver is now more robust to acquisition errors on all distros. It should automatically reconnect to the camera. Tested on Rolling and Humble.
However, because the Jazzy (and I believe Kilted) image transport does not support lifecycle nodes, the lifecycle feature is only available on Rolling.

@berndpfrommer
berndpfrommer merged commit 738833f into humble-devel Oct 30, 2025
6 checks passed
@berndpfrommer
berndpfrommer deleted the support_lifecycle_node branch October 30, 2025 16:21
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