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

Remove publisher #10

Closed
wants to merge 3 commits into from
Closed

Remove publisher #10

wants to merge 3 commits into from

Conversation

justinscorringe
Copy link
Collaborator

Adds RemovePublisher function to node which calls node.Shutdown and removes publisher from node.publishers

justinscorringe and others added 3 commits November 19, 2019 14:03
Added libtest_listener which tests if a subscriber node can be created on the ros system
Added libtest_pubsub which creates a publisher and a subscriber node, and tests if the message is sent and recieved correctly
Moved diagnostic tests into separate function in thinger
Split comment line into two in libtest_talker
@edwinhayes edwinhayes closed this Nov 19, 2019
@edwinhayes edwinhayes deleted the RemovePublisher branch November 19, 2019 23:35
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