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

Adding a service to set the image topic. #17

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

AndyZe
Copy link

@AndyZe AndyZe commented Oct 8, 2018

No description provided.

@AndyZe
Copy link
Author

AndyZe commented Oct 9, 2018

Sorry for all the tiny commits. I'll squash them if I can get past Jenkins.

@AndyZe
Copy link
Author

AndyZe commented Nov 5, 2018

ping @dirk-thomas

Copy link
Contributor

@dirk-thomas dirk-thomas left a comment

Choose a reason for hiding this comment

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

I am sorry for the late reply. I will hopefully get this released as soon as you had a chance to address the feedback. Thanks.

package.xml Outdated Show resolved Hide resolved
package.xml Outdated Show resolved Hide resolved
package.xml Outdated Show resolved Hide resolved
CMakeLists.txt Outdated Show resolved Hide resolved
CMakeLists.txt Outdated Show resolved Hide resolved
CMakeLists.txt Outdated Show resolved Hide resolved
src/rqt_image_view/image_view.cpp Outdated Show resolved Hide resolved
src/rqt_image_view/image_view.cpp Outdated Show resolved Hide resolved
src/rqt_image_view/image_view.cpp Outdated Show resolved Hide resolved
srv/SetImageTopic.srv Outdated Show resolved Hide resolved
@dirk-thomas
Copy link
Contributor

Numerous of my previous comments have been marked as resolved but not actually been addressed?

@AndyZe
Copy link
Author

AndyZe commented Apr 13, 2019

Sorry, I'm not sure how I missed so many comments

@MatthijsBurgh
Copy link
Contributor

This PR is close to being 3 years untouched.

I also think is overrated to control a GUI by a service. We have a gui, when we don't want command line.

@mabelzhang I suggest you close this PR

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.

4 participants