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.

Sign up for GitHub

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

Jump to bottom

Update REP-I0001 with implementation-related changes/clarifications #13

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update REP-I0001 with implementation-related changes/clarifications #13

wants to merge 2 commits into from

Commits on Jun 9, 2015

  1. Update REP-I0001 with implementation-related changes/clarifications 

     - add missing fields to Dynamic_Joint_State, to match ROS API
 - reorder/add fields in Dynamic_Group_Status, to match existing RobotStatus msg
 - rework configuration parameter, to minimize duplication
 - elaborate on operation, configuration, usage, and backwards compatibility
    @JeremyZoss
    JeremyZoss committed Jun 9, 2015
    Configuration menu
    Copy the full SHA
    378783f View commit details
    Browse the repository at this point in the history

Commits on Jun 12, 2015

  1. Clarification updates, per ros-industrial#13 review comments 

     - update header lines to meet RST spec
 - re-add unique joint-name assumption
 - fix literal-text to RST syntax
 - misc spaces and typos
 - add example for namespace-mapping config/behavior
 - add more discussion of `valid_fields` feature
    @JeremyZoss
    JeremyZoss committed Jun 12, 2015
    Configuration menu
    Copy the full SHA
    af8a3ca View commit details
    Browse the repository at this point in the history