Hard coded timeouts in camera driver decrease maximum FPS about pylon-ros-camera

[pablo-quilez]

Hello,

you are right, we plan to do it configurable. It is a trade off between CPU performance and publishing speed which is really depending on the computer you are using. Therefore it is tricky to change it in embedded / low performance systems.

I see two possibilities to address this issue:

• Create a service to set it (preferred)
• Set as ROS parameter (through our drag&bot module yaml https://github.com/basler/pylon-ros-camera/blob/master/pylon_camera/module_config/module.yaml)

Please if you want to collaborate you are very welcome! I think it is important to have this service for setting the timeouts, what do you think?

Thanks!

[claudiofantacci]

Hi @pablo-quilez thanks for reaching out :-)

I brainstormed with colleagues and we believe that a normal node-level ROS param, appearing also in this YAML configuration file, is the preferred solution.
Otherwise, we believe a service will be the second best option.

[pablo-quilez]

You are welcome @claudiofantacci ! It is a pleasure. I am planning to address all of these FPS related topics starting on the next week but maybe you can help me to define exactly which next steps should we do. I suggest:

1. Make timeouts configurable (TBD: param / services)
2. Allow both self triggering / software triggering in the camera
3. Increase the performance of building ROS blob from camera picture in memory. The way the driver currently extracts the picture from the memory and converts to ROS may not be so optimal.

Do you think this will be enough? Am I missing something?

[claudiofantacci]

Hey @pablo-quilez I think these 3 points are well taken.

1. This is important as we discussed previously
2. This is probably the most important. We cannot achieve good framerate and stability with self-triggering and we opted for software triggering. This point, together with 1. will make a huge improvement in the driver 🚀
3. Not sure about this atm. But if there can be improvements on the memory side I think it is worth spending some effort, especially for hi-res streams.

I'll brainstorm with other people next week and be back to you with some other comments if any 😄 Thanks again!

[pablo-quilez]

Hi!

We have added the grabbing and timeout parameters both as parameters and as services. After internal tests we have merged into the main devel branch. Hopefully we can see this issue as closed!

#55

@claudiofantacci please let me know if we are missing something. I can reopen the issue if necessary. Thanks!