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

Fix deprecation warning in paramater declaration #582

Merged
merged 1 commit into from
Sep 3, 2024

Conversation

kumar-sanjeeev
Copy link
Contributor

Updated pos2 and pos4 values from int to float as type implicitly mentioned in parameter declaration as rclpy.Parameter.Type.DOUBLE_ARRAY in publisher_forward_position_controller and publisher_joint_trajectory_controller.

Fixes: #1239 in ros2_controllers

Updated `pos2` and `pos4` values from int to float as type implicitly
mentioned in parameter declaration as `rclpy.Parameter.Type.DOUBLE_ARRAY` in `publisher_forward_position_controller` and `publisher_joint_trajectory_controller`.

Fixes: #1239
Copy link
Member

@bmagyar bmagyar left a comment

Choose a reason for hiding this comment

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

Thank you!

@bmagyar bmagyar merged commit 25c1f59 into ros-controls:master Sep 3, 2024
14 checks passed
Copy link
Contributor

@christophfroehlich christophfroehlich left a comment

Choose a reason for hiding this comment

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

Thanks!

@christophfroehlich
Copy link
Contributor

@Mergifyio backport humble iron

Copy link
Contributor

mergify bot commented Sep 5, 2024

backport humble iron

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Sep 5, 2024
mergify bot pushed a commit that referenced this pull request Sep 5, 2024
saikishor pushed a commit that referenced this pull request Sep 5, 2024
(cherry picked from commit 25c1f59)

Co-authored-by: Sanjeev <62834697+kumar-sanjeeev@users.noreply.github.com>
saikishor pushed a commit that referenced this pull request Sep 5, 2024
(cherry picked from commit 25c1f59)

Co-authored-by: Sanjeev <62834697+kumar-sanjeeev@users.noreply.github.com>
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