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

Add stereo preview example #895

Closed
wants to merge 2 commits into from
Closed

Conversation

will-v-pi
Copy link
Collaborator

Add an example demonstrating the use of stride, where the stride is set to twice the image width to allow an image from a second camera to be copied into the empty space, creating a stereo image

Add an example demonstrating the use of stride, where the stride is set
to twice the image width to allow an image from a second camera to be
copied into the empty space, creating a stereo image

Signed-off-by: William Vinnicombe <william.vinnicombe@raspberrypi.com>
Add a test to check the stride is being set correctly when provided in
the stream configuration

Signed-off-by: William Vinnicombe <william.vinnicombe@raspberrypi.com>
@6by9
Copy link
Contributor

6by9 commented Apr 17, 2024

On vc4 at least we could allow the selection API to set the compose rectangle to cover the part of the double sized buffer that the ISP is meant to write into. The same buffer could therefore be written by 2 instances of libcamera (one for each eye). The offset will need to be aligned based on the ISP constraints.

I have no idea with regard how to pass that information through libcamera in order to request this though!

@davidplowman
Copy link
Collaborator

@will-v-pi There was some kernel update required which is why we didn't merge this immediately, do we think that might all be sorted now? Perhaps it's worth rebasing and re-pushing...?

@davidplowman
Copy link
Collaborator

Closing and re-opening just to re-run the tests.

@davidplowman
Copy link
Collaborator

This has now been rebased and merged here: #1146

Sorry it took so long!!

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.

3 participants