-
Notifications
You must be signed in to change notification settings - Fork 120
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
reconfigurable transport scoped parameters for theora_image_transport
- runtime reconfiguration for publisher and subscriber - <image>.<transport>.<param> as future replacement of <image>.<param> - e.g. `image.theora.target_bitrate` instead of `image.target_bitrate` - support both ways for now - emit deprecation warnings on setting through non-transport scoped parameter - synchronize deprecated changes to new ones - this is necessary for dynamic reconfigure The publisher is a bit more complex here - in general we reload config on publish - but some code paths (conditional compilation) - would result in resetting context on every config reload - so we flag to reload config only on parameter event change - and reload it once on init to mimic ROS1 dynamic reconfigure setup The subscriber is simple as it already had necessary checks. Related to #140 (cherry picked from commit b7d409f) # Conflicts: # theora_image_transport/src/theora_publisher.cpp # theora_image_transport/src/theora_subscriber.cpp
- Loading branch information
1 parent
29e29e3
commit 56d4a65
Showing
2 changed files
with
24 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters