[Humble] Don't waste time on blocking image data readout about pylon-ros-camera HOT 4 OPEN

Hello @FrancoisPicardDTI,

I can see why it might not be a high priority as in any case one could do an asynchronous action request to take images which ineherently is non blocking. In my specific case, the camera position moves continuously which is why I need to make sure the exposure is finished before moving furthur. But well that can also be done (my current solution) with a simple waiting for some safety time (~7us) after sending the action request.

But personally for me a clean solution would be getting a callback on when the exposure is finished to know the exact times. I have thought about it and it is not that simple because the event handlers are registered for all the grab events (low level implementation) whereas it needs to be intertwined with the action request to react on a specifc request. I thought about a solution like this:

• Send an action request for grabbing one image.
• forward the goal handle of the request to the event handler (maybe with a lambda function or something)
• Upon finishing exposure, publish a feedback for the action request indicating exposure was finished.
• Upon finishing readout, return the result.

This going up and down all the way from ROS interface to camera implementation and back, can be a bit tricky. I would like to work on it of course but at the moment my resources are limited. If I get some free time I will look at it.

from pylon-ros-camera.

@alireza-moayyedi Hi, yes, your thoughts are generally right & yes, the pylon C++ API does support both (1) working with timeouts (while the pylon engine is polling on the queue); and (2) using callbacks (registering for events). The latter is not implemented on ROS yet and despite I'm not the best ROS expert I'm not sure it would bring the same benefits as it does in the pure pylon C++ implementation.
What I'm trying to say is that you are all free to give it a try if you've got the time and believe your use case will gain advantages of that.
Cheers

from pylon-ros-camera.

Hello @alireza-moayyedi
To answer you, no, this feature is not implemented in this repository. Either because at the time, the API was not allowing it, or because it has been deemed not suitable for the current implementation.
And no, it is not planned for now. It can become a priority if several users are asking for it.
Are you planning to work on this feature?

from pylon-ros-camera.

I'm also interested in this feature but our application is not that time-critical so that the current implementation is OK for us.

from pylon-ros-camera.