rosparam_shortcuts icon indicating copy to clipboard operation
rosparam_shortcuts copied to clipboard

[Draft/Discussion] Recursive parameter interface

Open henningkayser opened this issue 4 years ago • 2 comments

@tylerjw here is a draft interface that could combine the parameter tree as implemented in ros2_params with the new NodeParameters interface. To me the only hurdle would be how callbacks would actually be handled in the parameter tree, or if we would only allow callbacks for top-level parameters and for the leafs (= node parameters).

henningkayser avatar Apr 29 '21 10:04 henningkayser

@henningkayser I made some small edits here: https://github.com/henningkayser/rosparam_shortcuts/pull/1

tylerjw avatar Apr 30 '21 18:04 tylerjw

Codecov Report

Merging #17 (af64edd) into ros2 (2c8412c) will not change coverage. The diff coverage is n/a.

:exclamation: Current head af64edd differs from pull request most recent head 1daaab6. Consider uploading reports for the commit 1daaab6 to get more accurate results Impacted file tree graph

@@           Coverage Diff           @@
##             ros2      #17   +/-   ##
=======================================
  Coverage   22.53%   22.53%           
=======================================
  Files           3        3           
  Lines         142      142           
=======================================
  Hits           32       32           
  Misses        110      110           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 2c8412c...1daaab6. Read the comment docs.

codecov[bot] avatar May 05 '21 12:05 codecov[bot]