Skip to content
This repository was archived by the owner on Oct 17, 2025. It is now read-only.

Galactic: Pass time and period to update function#88

Merged
ahcorde merged 3 commits intoros-controls:masterfrom
bmagyar:update-update-signature-for-galactic
Oct 26, 2021
Merged

Galactic: Pass time and period to update function#88
ahcorde merged 3 commits intoros-controls:masterfrom
bmagyar:update-update-signature-for-galactic

Conversation

@bmagyar
Copy link
Copy Markdown
Member

@bmagyar bmagyar commented Sep 18, 2021

@bmagyar bmagyar changed the title Pass time and period to update function Galactic: Pass time and period to update function Sep 18, 2021
Copy link
Copy Markdown
Member

@destogl destogl left a comment

Choose a reason for hiding this comment

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

Looking good as per update on controller manager.

Copy link
Copy Markdown
Collaborator

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

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

Hi @bmagyar, I created a branch for foxy. Do you mind to update gazebo_ros2_control/.github/workflows/ to target galactic?

@bmagyar
Copy link
Copy Markdown
Member Author

bmagyar commented Sep 21, 2021

sure no problem

@bmagyar
Copy link
Copy Markdown
Member Author

bmagyar commented Oct 15, 2021

@ahcorde could you please kick the CI to retry? The packages should be available from upstream now

Copy link
Copy Markdown
Collaborator

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

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

Do you mind to change this line in the gazebo_ros2_control/package.xml

  <exec_depend>joint_state_controller</exec_depend>

with

  <exec_depend>joint_state_broadcaster</exec_depend>

@bmagyar
Copy link
Copy Markdown
Member Author

bmagyar commented Oct 15, 2021

Done!

@ahcorde
Copy link
Copy Markdown
Collaborator

ahcorde commented Oct 26, 2021

I will fix CI in other PR

@ahcorde ahcorde merged commit 83089d3 into ros-controls:master Oct 26, 2021
ksotebeer pushed a commit to ksotebeer/gazebo_ros2_control that referenced this pull request Jun 14, 2022
* Pass time and period to update function

* Use Galactic in GH action

* joint_state_broadcaster in town
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants