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

Use node to create clock used to stamp publications #407

Merged
merged 1 commit into from
Jun 11, 2019
Merged

Use node to create clock used to stamp publications #407

merged 1 commit into from
Jun 11, 2019

Conversation

cottsay
Copy link
Member

@cottsay cottsay commented May 28, 2019

Existing behavior is that initialpose is always published with the wall time, even when use_sim_time is true.

I'm hoping there is a better way to get a hold of the ROS clock - let me know and I'll update this PR.

@cottsay cottsay added the enhancement New feature or request label May 28, 2019
@cottsay cottsay self-assigned this May 28, 2019
@cottsay cottsay added this to Proposed in Dashing Patch Release 1 via automation Jun 10, 2019
@cottsay cottsay changed the title Use ROS time when publishing to initialpose Use node to create clock used to stamp publications Jun 10, 2019
@cottsay
Copy link
Member Author

cottsay commented Jun 10, 2019

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@wjwwood wjwwood merged commit 58f0cc1 into ros2:ros2 Jun 11, 2019
Dashing Patch Release 1 automation moved this from Proposed to Needs Backport Jun 11, 2019
@clalancette clalancette moved this from Needs Backport to Needs Release in Dashing Patch Release 1 Jun 12, 2019
@clalancette clalancette moved this from Needs Release to Released in Dashing Patch Release 1 Jun 12, 2019
@cottsay cottsay deleted the ros_time branch July 31, 2019 00:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants