-
Notifications
You must be signed in to change notification settings - Fork 184
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
Fix opencv_mertems_merge example #15
Conversation
Add possibility to position the upper left hand corner of the preview window.
Also update drm preview example
It seems picam2.configure has to be called before each picam2.start. Without this the exposure short already aborts.
Hi, thanks for looking at this. I'm a bit bothered by it because reconfiguring really shouldn't be necessary, and it causes an entire teardown of the camera session, re-allocation of all the buffers and so on. Is this change fixing some warnings (but the code works anyway, which is the case for me) or that something is genuinely failing? Thanks! |
Here is the output I get from the original version:
It is hanging in the 'short' capture |
I'll try it over vnc but I don't think that should make a difference. From what you've said previously I take it this is a 2GB Pi 4B, and a v1 camera, so I'll try to match that as well. But in any case, I don't think the change here is the correct fix, could you perhaps file a bug instead? Thanks! |
Just tried this with the v1 sensor and indeed that seems to be the problem. Almost certainly a sensor/driver issue, so we'll have to look into that. |
Yes it is a 2G Pi4B and a V1 camera. |
Tried also with the HQ camera on Pi 4B 2G with 32-bit OS.
|
Closed transferred to issue #18 |
It seems picam2.configure has to be called before each picam2.start.
Without this the exposure short already aborts.