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.

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

[Feature-Request] Support autoexposure with maximum exposure duration #613

Open
abaeyens-imod opened this issue Oct 24, 2024 · 3 comments
Open
Labels
enhancement New feature or request

Comments

@abaeyens-imod
Copy link

abaeyens-imod commented Oct 24, 2024

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.

@abaeyens-imod abaeyens-imod added the enhancement New feature or request label Oct 24, 2024
@Serafadam
Copy link
Collaborator

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

@abaeyens-imod
Copy link
Author

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

@abaeyens-imod
Copy link
Author

Tested together with #614, works well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants