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

[service introspection] Add ServiceEvent, ServiceEventInfo, ServiceEventType messages #141

Closed
wants to merge 3 commits into from

Conversation

ihasdapie
Copy link
Member

@ihasdapie ihasdapie commented Jun 29, 2022

This PR is a prototype for the service introspection REP (ros-infrastructure/rep#360) implementing the proposed single serialized message approach (ros-infrastructure/rep#360 (comment)).

For instructions on how to build and run this see the meta-ticket ros2/ros2#1285.

Resolves ros2/ros2#1285

@ihasdapie ihasdapie changed the title Add Service Introspection messages [service introspection] Add ServiceEvent, ServiceEventInfo, ServiceEventType messages Jun 29, 2022
@ihasdapie ihasdapie mentioned this pull request Jun 29, 2022
14 tasks
Copy link
Member

@ivanpauno ivanpauno left a comment

Choose a reason for hiding this comment

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

LGTM!

@ihasdapie ihasdapie self-assigned this Jul 14, 2022
@ihasdapie
Copy link
Member Author

Closing this PR in lieu of #143

@ihasdapie ihasdapie closed this Jul 27, 2022
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.

Service Introspection
3 participants