-
Notifications
You must be signed in to change notification settings - Fork 190
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
PanoramaInfo msg can have image height and width? #2752
Comments
I didn't design PanoramaInfo without Image size since it is redundant, but it sounds good idea to put it in PanoramaInfo. |
Thank you, I will make a PR. |
@knorth55 Let's make it simple. Please remove a subscriber of panorama image. |
@sktometometo OK, I will |
i made a PR #2753 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
rects_array_in_panorama_to_bounding_box_array
node subscribe full panorama image,but it uses only image height and image width image.
https://github.com/jsk-ros-pkg/jsk_recognition/blob/master/jsk_perception/node_scripts/rect_array_in_panorama_to_bounding_box_array.py#L53-L68
But when the network connection is low, it is hard to subscribe full panorama image.
We thought
PanoramaInfo
msg can haveimage
height andimage
width information@sktometometo what do you think?
cc. @AyumuIwa
The text was updated successfully, but these errors were encountered: