Skip to content

Add camera functions for exposure time and gain increment.#1043

Merged
EmmanuelP merged 8 commits into
AravisProject:mainfrom
feuerste:exposure_gain_increment
Nov 11, 2025
Merged

Add camera functions for exposure time and gain increment.#1043
EmmanuelP merged 8 commits into
AravisProject:mainfrom
feuerste:exposure_gain_increment

Conversation

@feuerste

@feuerste feuerste commented Oct 28, 2025

Copy link
Copy Markdown
Contributor

This is in particular useful for a Basler acA1920-25gc, where arbitrary exposure times can be set and read, but only multiples of the correct increment are actually applied by the camera driver.

Also the viewer has been adjusted to only allow multiples of the increment.

This was tested with an Allied Vision Alvium G1-1240c, FLIR Blackfly S BFS-PGE-23S3C, and 3 different Basler cameras.

Comment thread src/arvcamera.c
@feuerste feuerste changed the title Add functions for exposure time and gain increment. Add camera functions for exposure time and gain increment. Oct 28, 2025
@EmmanuelP EmmanuelP marked this pull request as draft October 29, 2025 14:44
@EmmanuelP

Copy link
Copy Markdown
Contributor

Thanks for your work on this, Marco.

Please mark this merge request ready for review when you think it is.

@feuerste feuerste marked this pull request as ready for review October 29, 2025 15:02
@feuerste feuerste marked this pull request as draft October 29, 2025 15:59
@feuerste feuerste marked this pull request as ready for review October 29, 2025 16:47
@feuerste feuerste marked this pull request as draft October 30, 2025 07:39
@feuerste feuerste marked this pull request as ready for review October 30, 2025 07:52
@feuerste

Copy link
Copy Markdown
Contributor Author

Thanks for your work on this, Marco.

Please mark this merge request ready for review when you think it is.

Thanks Emmanuel! The PR is ready for review now, I also tested it with a couple of cameras.

@feuerste

Copy link
Copy Markdown
Contributor Author

@EmmanuelP Would you be available for a review? Thanks a lot!

@EmmanuelP

Copy link
Copy Markdown
Contributor

Hi @feuerste , sorry for the delay. I will try to review your work and test it with my device collection soon...

@EmmanuelP EmmanuelP merged commit 85e5a2f into AravisProject:main Nov 11, 2025
5 checks passed
@EmmanuelP

Copy link
Copy Markdown
Contributor

Thanks.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants