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

ROS Service Enhancement for the Single Image Detector Node #86

Open
swiz23 opened this issue Jan 12, 2021 · 1 comment
Open

ROS Service Enhancement for the Single Image Detector Node #86

swiz23 opened this issue Jan 12, 2021 · 1 comment

Comments

@swiz23
Copy link

swiz23 commented Jan 12, 2021

Could there be another type of Service where instead of pointing to a picture file, you put the actual picture in the Service message as a sensor_msgs/Image type?

For my application, all images are coming from a ROS topic that my node subscribes too. It seems more intuitive to just stick the latest image into the Service message than to save the image to a file and pass the file location to the single image detector node. I'd rather not use the continuous detector node since I don't want the overhead, and my application only needs to get a single shot of the AR tag.

Thanks

@christian-rauch
Copy link
Contributor

I agree that the implementation of the single image detector is weird and its usage is cumbersome. For analysing a single image file, using a standalone executable (with ROS etc.) that reads the image file from disk and writes the result to stdout would be more appropriate.

I would suggest removing the ROS service for analysing images from files and just provide an executable or python script in the apriltag repo for this task.

@swiz23 A workaround is to send single non-continuous images via a different topic. This way you avoid the continuous detection but can keep the service call behaviour. You just have to synchronise the request and the response by the timestamp in the headers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants