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

transition events don't populate transition id, label, or timestamp #1019

Open
Aposhian opened this issue Nov 10, 2022 · 0 comments · May be fixed by #1140
Open

transition events don't populate transition id, label, or timestamp #1019

Aposhian opened this issue Nov 10, 2022 · 0 comments · May be fixed by #1140

Comments

@Aposhian
Copy link

Aposhian commented Nov 10, 2022

Bug report

Required Info:

  • Operating System:
    • N/A
  • Installation type:
    • binaries
  • Version or commit hash:
    • humble
  • DDS implementation:
    • CycloneDDS
  • Client library (if applicable):
    • N/A

Steps to reproduce issue

Not all the fields of lifecycle_msgs/msg/TransitionEvent are being utilized

ros2 topic echo /<node>/transition_event lifecycle_msgs/msg/TransitionEvent

Expected behavior

timestamp: 1668097069451777251
transition:
  id: 1
  label: configure
start_state:
  id: 1
  label: unconfigured
goal_state:
  id: 10
  label: configuring

Actual behavior

timestamp: 0
transition:
  id: 0
  label: ''
start_state:
  id: 1
  label: unconfigured
goal_state:
  id: 10
  label: configuring

Additional information

This matters for things like launch_ros.event_handlers.OnStateTransition which present an interface to match on transition, but which will fail every time.

@CursedRock17 CursedRock17 linked a pull request Mar 26, 2024 that will close this issue
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 a pull request may close this issue.

1 participant