Skip to content
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

Support ROS parameter descriptions #68

Merged
merged 3 commits into from
Oct 5, 2023

Commits on Sep 26, 2023

  1. Minimal implementation of GetParameterTypes and DescribeParameters se…

    …rvices
    
    With this, the parameters are shown in Rqt or Foxglove Studio.
    wentasah committed Sep 26, 2023
    Configuration menu
    Copy the full SHA
    48423f0 View commit details
    Browse the repository at this point in the history

Commits on Oct 2, 2023

  1. r2r_macros: Remove TODO

    The code is already merged, so renaming makes little sense.
    wentasah committed Oct 2, 2023
    Configuration menu
    Copy the full SHA
    437bd41 View commit details
    Browse the repository at this point in the history
  2. Derive ROS parameter description from field doc comments

    With this change, adding doc comments to fields of structures used
    with `#[derive(RosParams)]` results in those comments being used as
    parameter description.
    
    See r2r/examples/parameters_derive.rs for how to use and test this
    feature.
    
    *BREAKING CHANGE*
    
    This commit changes r2r public API. Previously Node::params contained
    HashMap<String, ParameterValue>, now it contains HashMap<String, Parameter>.
    
    If you previously used the ParameterValue from this HashMap, now you
    can get the same by using the .value field of the Parameter structure.
    wentasah committed Oct 2, 2023
    Configuration menu
    Copy the full SHA
    86d1605 View commit details
    Browse the repository at this point in the history