Skip to content

Publish Foreman status over ROS topic - #21

Open
VitezGabriela wants to merge 4 commits into
ros-controls:masterfrom
b-robotized-forks:feat/status-topic
Open

Publish Foreman status over ROS topic#21
VitezGabriela wants to merge 4 commits into
ros-controls:masterfrom
b-robotized-forks:feat/status-topic

Conversation

@VitezGabriela

Copy link
Copy Markdown

Summary

  • Add ForemanStatus and ForemanErrorState ROS 2 messages.
  • Add RosStatusPublisher to publish Foreman's latest engine snapshot on /foreman/status.
  • Add unit tests

Behavior Change

Foreman now publishes a status topic at:

/foreman/status

The topic contains:

string goal
bool ready
bool at_goal
ForemanErrorState error

Testing

colcon build --packages-up-to foreman
colcon test --packages-up-to foreman
colcon test-result --verbose

Result: 42 tests, 0 errors, 0 failures, 0 skipped

Comment thread foreman/foreman/adapters/ros_status_publisher.py Outdated
Comment thread foreman/foreman/adapters/ros_status_publisher.py Outdated
Comment thread foreman/foreman/adapters/ros_status_publisher.py Outdated
Comment thread foreman/foreman/adapters/ros_status_publisher.py Outdated
Comment thread foreman/foreman/adapters/ros_status_publisher.py Outdated
Comment thread foreman/foreman/adapters/ros_status_publisher.py Outdated
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.

2 participants