Skip to content

add read_only parameters to diff_drive_controller #1781

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
+36 −0
Open

add read_only parameters to diff_drive_controller #1781

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
c76d084
add read_only parameters in diff_drive_controller
adhithiyan110 Jun 30, 2025
6d210f4
Merge branch 'ros-controls:master' into master
adhithiyan110 Jul 6, 2025
88404fd
Refactor diff_drive_controller parameter config
adhithiyan110 Jul 6, 2025
a138243
Merge branch 'master' into master
christophfroehlich Jul 7, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 36 additions & 0 deletions diff_drive_controller/src/diff_drive_controller_parameter.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ diff_drive_controller:
type: string_array,
default_value: [],
description: "Names of the left side wheels' joints",
read_only: true,
validation: {
not_empty<>: []
}
Expand All @@ -11,6 +12,7 @@ diff_drive_controller:
type: string_array,
default_value: [],
description: "Names of the right side wheels' joints",
read_only: true,
validation: {
not_empty<>: []
}
Expand All @@ -19,6 +21,7 @@ diff_drive_controller:
type: double,
default_value: 0.0,
description: "Shortest distance between the left and right wheels. If this parameter is wrong, the robot will not behave correctly in curves.",
read_only: true,
validation: {
gt<>: [0.0]
}
Expand All @@ -27,6 +30,7 @@ diff_drive_controller:
type: double,
default_value: 0.0,
description: "Radius of a wheel, i.e., wheels size, used for transformation of linear velocity into wheel rotations. If this parameter is wrong the robot will move faster or slower then expected.",
read_only: true,
validation: {
gt<>: [0.0]
}
Expand All @@ -35,94 +39,112 @@ diff_drive_controller:
type: double,
default_value: 1.0,
description: "Correction factor for wheel separation (TODO(destogl): Please help me describe this correctly)",
read_only: true,
}
left_wheel_radius_multiplier: {
type: double,
default_value: 1.0,
description: "Correction factor when radius of left wheels differs from the nominal value in ``wheel_radius`` parameter.",
read_only: true,
}
right_wheel_radius_multiplier: {
type: double,
default_value: 1.0,
description: "Correction factor when radius of right wheels differs from the nominal value in ``wheel_radius`` parameter.",
read_only: true,
}
tf_frame_prefix_enable: {
type: bool,
default_value: true,
description: "Enables or disables appending tf_prefix to tf frame id's.",
read_only: true,
}
tf_frame_prefix: {
type: string,
default_value: "",
description: "(optional) Prefix to be appended to the tf frames, will be added to odom_id and base_frame_id before publishing. If the parameter is empty, controller's namespace will be used.",
read_only: true,
}
odom_frame_id: {
type: string,
default_value: "odom",
description: "Name of the frame for odometry. This frame is parent of ``base_frame_id`` when controller publishes odometry.",
read_only: true,
}
base_frame_id: {
type: string,
default_value: "base_link",
description: "Name of the robot's base frame that is child of the odometry frame.",
read_only: true,
}
pose_covariance_diagonal: {
type: double_array,
default_value: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0],
description: "Odometry covariance for the encoder output of the robot for the pose. These values should be tuned to your robot's sample odometry data, but these values are a good place to start: ``[0.001, 0.001, 0.001, 0.001, 0.001, 0.01]``.",
read_only: true,
}
twist_covariance_diagonal: {
type: double_array,
default_value: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0],
description: "Odometry covariance for the encoder output of the robot for the speed. These values should be tuned to your robot's sample odometry data, but these values are a good place to start: ``[0.001, 0.001, 0.001, 0.001, 0.001, 0.01]``.",
read_only: true,
}
open_loop: {
type: bool,
default_value: false,
description: "If set to true the odometry of the robot will be calculated from the commanded values and not from feedback.",
read_only: false,
Copy link
Member

Choose a reason for hiding this comment

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

Do we want this to be false?. Does it make sense to change this online?

Copy link
Author

Choose a reason for hiding this comment

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

can i make this true?

Copy link
Member

Choose a reason for hiding this comment

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

Sure

}
position_feedback: {
type: bool,
default_value: true,
description: "Is there position feedback from hardware.",
read_only: false,
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
read_only: false,
read_only: true,

}
enable_odom_tf: {
type: bool,
default_value: true,
description: "Publish transformation between ``odom_frame_id`` and ``base_frame_id``.",
read_only: false,
}
cmd_vel_timeout: {
type: double,
default_value: 0.5, # seconds
description: "Timeout in seconds, after which input command on ``cmd_vel`` topic is considered staled.",
read_only: false,
}
publish_limited_velocity: {
type: bool,
default_value: false,
description: "Publish limited velocity value.",
read_only: false,
}
velocity_rolling_window_size: {
type: int,
default_value: 10,
description: "Size of the rolling window for calculation of mean velocity use in odometry.",
read_only: true,
}
publish_rate: {
type: double,
default_value: 50.0, # Hz
description: "Publishing rate (Hz) of the odometry and TF messages.",
read_only: false,
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
read_only: false,
read_only: true,

This should be constant too

Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
read_only: false,
read_only: true,

This should be constant too

}
linear:
x:
max_velocity: {
type: double,
default_value: .NAN,
read_only: false,
validation: {
"control_filters::gt_eq_or_nan<>": [0.0]
}
}
min_velocity: {
type: double,
default_value: .NAN,
read_only: false,
validation: {
"control_filters::lt_eq_or_nan<>": [0.0]
}
Expand All @@ -131,6 +153,7 @@ diff_drive_controller:
type: double,
default_value: .NAN,
description: "Maximum acceleration in forward direction.",
read_only: false,
validation: {
"control_filters::gt_eq_or_nan<>": [0.0]
}
Expand All @@ -139,6 +162,7 @@ diff_drive_controller:
type: double,
default_value: .NAN,
description: "Maximum deceleration in forward direction.",
read_only: false,
validation: {
"control_filters::lt_eq_or_nan<>": [0.0]
}
Expand All @@ -147,6 +171,7 @@ diff_drive_controller:
type: double,
default_value: .NAN,
description: "Maximum acceleration in reverse direction. If not set, -max_acceleration will be used.",
read_only: false,
validation: {
"control_filters::lt_eq_or_nan<>": [0.0]
}
Expand All @@ -155,20 +180,23 @@ diff_drive_controller:
type: double,
default_value: .NAN,
description: "Maximum deceleration in reverse direction. If not set, -max_deceleration will be used.",
read_only: false,
validation: {
"control_filters::gt_eq_or_nan<>": [0.0]
}
}
max_jerk: {
type: double,
default_value: .NAN,
read_only: false,
validation: {
"control_filters::gt_eq_or_nan<>": [0.0]
}
}
min_jerk: {
type: double,
default_value: .NAN,
read_only: false,
validation: {
"control_filters::lt_eq_or_nan<>": [0.0]
}
Expand All @@ -178,13 +206,15 @@ diff_drive_controller:
max_velocity: {
type: double,
default_value: .NAN,
read_only: false,
validation: {
"control_filters::gt_eq_or_nan<>": [0.0]
}
}
min_velocity: {
type: double,
default_value: .NAN,
read_only: false,
validation: {
"control_filters::lt_eq_or_nan<>": [0.0]
}
Expand All @@ -193,6 +223,7 @@ diff_drive_controller:
type: double,
default_value: .NAN,
description: "Maximum acceleration in positive direction.",
read_only: false,
validation: {
"control_filters::gt_eq_or_nan<>": [0.0]
}
Expand All @@ -201,6 +232,7 @@ diff_drive_controller:
type: double,
default_value: .NAN,
description: "Maximum deceleration in positive direction.",
read_only: false,
validation: {
"control_filters::lt_eq_or_nan<>": [0.0]
}
Expand All @@ -209,6 +241,7 @@ diff_drive_controller:
type: double,
default_value: .NAN,
description: "Maximum acceleration in negative direction. If not set, -max_acceleration will be used.",
read_only: false,
validation: {
"control_filters::lt_eq_or_nan<>": [0.0]
}
Expand All @@ -217,20 +250,23 @@ diff_drive_controller:
type: double,
default_value: .NAN,
description: "Maximum deceleration in negative direction. If not set, -max_deceleration will be used.",
read_only: false,
validation: {
"control_filters::gt_eq_or_nan<>": [0.0]
}
}
max_jerk: {
type: double,
default_value: .NAN,
read_only: false,
validation: {
"control_filters::gt_eq_or_nan<>": [0.0]
}
}
min_jerk: {
type: double,
default_value: .NAN,
read_only: false,
validation: {
"control_filters::lt_eq_or_nan<>": [0.0]
}
Expand Down
Loading