Skip to content

Conversation

@aalmrad
Copy link

@aalmrad aalmrad commented Oct 30, 2025

Modified the pid terms in the pid_ros.cpp file after receiving build errors.

@mergify
Copy link

mergify bot commented Oct 30, 2025

@aalmrad, all pull requests must be targeted towards the master development branch.
Once merged into master, it is possible to backport to jazzy, but it must be in master
to have these changes reflected into new distributions.

@aalmrad
Copy link
Author

aalmrad commented Oct 30, 2025

The modifications were tested only in Jazzy

Copy link
Member

@saikishor saikishor left a comment

Choose a reason for hiding this comment

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

@aalmrad you can only create the PRs against rolling and then we will backport them later

@christophfroehlich
Copy link
Contributor

Which build errors did you get? The message definition changed, have you used the correct version of control_msgs? (jazzy branch, see README)

@aalmrad
Copy link
Author

aalmrad commented Nov 4, 2025

@christophfroehlich , jazzy branch of control_msgs solved the issue. Thanks.

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