Skip to content

app_full.py is compatible with monochrome sensors. Fixes #1380#1383

Closed
fiepfiep wants to merge 2 commits intoraspberrypi:nextfrom
fiepfiep:app_full_mono
Closed

app_full.py is compatible with monochrome sensors. Fixes #1380#1383
fiepfiep wants to merge 2 commits intoraspberrypi:nextfrom
fiepfiep:app_full_mono

Conversation

@fiepfiep
Copy link
Copy Markdown

@fiepfiep fiepfiep commented May 5, 2026

Fixes #1380
add a check for monochrome sensors before accessing controls such as AWB, saturation.
@davidplowman

@davidplowman davidplowman changed the base branch from main to next May 6, 2026 06:49
@davidplowman
Copy link
Copy Markdown
Collaborator

Thanks for the PR. Actually we've just added a mono sensor into our "fleet" of test devices which is why there are some failures, so we'll get those cleaned up first.

On reflection, I might add a picam2.is_mono() function, which might make this a bit neater. I'll also replace 5 by libcamera.properties.draft.ColorFilterArrangementEnum.MONO which, although a bit of a mouthful, doesn't have that fragility problem we mentioned elsewhere. Thanks!

@davidplowman
Copy link
Copy Markdown
Collaborator

We've gone through all the other test failures for mono sensors and fixed them up, and taken care of this one too. That's all been merged onto our next branch now so will be in the next release, along with a picam2.is_mono property. I think that takes care of everything so I'll close this one now, but thanks for bringing it all to our attention. With a mono sensor in our test farm too now, hopefully we'll get less of this kind of thing in future!

@fiepfiep fiepfiep deleted the app_full_mono branch May 6, 2026 10:53
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.

[BUG] app_full.py example using monochrome sensor

3 participants