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.

Sign up for GitHub

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

Jump to bottom

convert line_detector to use image_transport or as a nodelet into the camera driver #22

Open
bsubei opened this issue Oct 25, 2014 · 7 comments
Open

convert line_detector to use image_transport or as a nodelet into the camera driver #22

bsubei opened this issue Oct 25, 2014 · 7 comments
Assignees
@bsubei
Labels
line_detection vision

Comments

@bsubei
Copy link
Contributor

bsubei commented Oct 25, 2014

huge performance gains to be had.
http://wiki.ros.org/image_transport
@bsubei bsubei self-assigned this Oct 25, 2014
@bsubei bsubei added the vision label Oct 25, 2014
@bsubei
Copy link
Contributor Author

bsubei commented Oct 26, 2014

turns out there is no Python support for image_transport ??? If true, then it will be necessary to convert line_detector to C++ after prototyping (in Python) is complete.

@bsubei
Copy link
Contributor Author

bsubei commented Oct 26, 2014

consider republish node as alternative for Python implementation

@bsubei
Copy link
Contributor Author

bsubei commented Oct 26, 2014

Maybe 4dfdee9 to subscribe to compressed image already takes care of this? No need for image_transport if I'm already subscribed to a compressed image (isn't that the whole point of image_transport?).

Not sure, so I posted a question on rosanswers.

@bsubei
Copy link
Contributor Author

bsubei commented Nov 9, 2014

this could be related to #27 dunno

@bethloosevelt
Copy link

What's the feasibility of having our python node call some C++ that can handle image_transport?

@bsubei bsubei changed the title convert line_detector to subscribe to images using image_transport convert line_detector to use image_transport or as a nodelet into the camera driver Jun 15, 2015
@bsubei
Copy link
Contributor Author

bsubei commented Jun 15, 2015

consider making line_detection into a nodelet injected into the camera driver (once it's converted to C++)

@l0g1x
Copy link
Member

l0g1x commented Nov 18, 2015

update on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bsubei bsubei

Labels
line_detection vision
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@l0g1x @bsubei @bethloosevelt