[Feature-Request] Support autoexposure with maximum exposure duration

abaeyens-imod commented 1 month ago

`why`

We and our clients use the Luxonis cameras on moving robots for VIO and VSLAM purposes. When we currently use the Luxonis cameras in dark environments, the exposure duration becomes too long resulting in severe motion blur, making the resulting images unusable (or at least result in severely degraded performance). Setting the exposure manually provides usable images for that lighting environment, but doesn't work in other environments.

Some other cameras, such as the Intel RealSense T265, don't offer this as a setting but do this automatically (presumably they have a suitable maximum exposure duration hardcoded).

`what`

Add a parameter max_exposure that sets a maximum exposure duration.
Autoexposure works as before, except that the ISO/sensitivity is increased as necessary to keep the exposure duration lower than max_exposure. It's fine if the image becomes noisy, and dark once the sensitivity has reached its maximum.
This is most important for the stereo pair, though also useful for the RGB camera.
In case it's not possible to make this adjustable, and that only one particular maximum exposure duration is supported, we're open to discussing what value would be suitable.
We would like to see this setting also for Noetic.

`how`

No idea, not familiar with the exact possibilities of these cameras.

Serafadam commented 4 weeks ago

Hi, please check this branch to see if this implementation works for your case. You can find the new parameters here

abaeyens-imod commented 4 weeks ago

Really appreciated, I'll test and report back early next week.

abaeyens-imod commented 3 weeks ago

Tested together with https://github.com/luxonis/depthai-ros/issues/614, works well.

luxonis / depthai-ros