Skip to content

Support exception in callback #30

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

Open
wants to merge 1 commit into
base: kinetic-devel
Choose a base branch
from

Conversation

furushchev
Copy link
Contributor

@furushchev furushchev commented Apr 30, 2025

This PR introduces improvements in exception handling during the invocation of registered callback functions. The changes ensure that exceptions thrown within callbacks are managed appropriately, leading to the following benefits:

  • Continuous Execution: If an exception occurs during the execution of a callback, the overall program does not halt and continues to execute.
  • Dynamic Reconfiguration Support: By allowing specific exceptions to be thrown within the callback during requests to change values by the Dynamic Reconfigure client, the system can notify of a failure of the changing to the client and rollback any changes as needed.

In addition to the changes, I fixed an issue on that the registerVariable did not reflect the initial value set by dynamic reconfigure (stored in rosparam).


The test case is also added.

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.

1 participant