Skip to content

Conversation

@Rahul-K-A
Copy link

Description

Original GSOC proposal here. Adds additional fields to the ParameterType and ParameterValue that allow storage of YAML strings

Fixes # (issue)

Is this user-facing behavior change?

Did you use Generative AI?

No

Additional Information

  1. Companion rcl PR : [Draft]Add structured parameter support rcl#1254
  2. Companion rclcpp PR: [Draft]Add structured parameter support rclcpp#2944
  3. Companion rclpy PR: [Draft]Add structured parameter support rclpy#1494

uint8 PARAMETER_INTEGER_ARRAY=7
uint8 PARAMETER_DOUBLE_ARRAY=8
uint8 PARAMETER_STRING_ARRAY=9
uint8 PARAMETER_YAML=5

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor issue: I would make the new value 10, in this way you have at least some backward compatibility...

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.

3 participants