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
Libcamera #697
Libcamera #697
Conversation
…ll of m_VideoApp and m_StillApp with m_CameraApp
Thanks! why is libcamera-apps included in the source? I think it should be used as an external dependency since it is a separate and fast moving project as well. |
It IS apparently a fast moving project, and I for my part would prefer a static end result instead of dealing with compile-from-source lib versions. When they stabilise or actually do anything new we actually need, we can use the libs again. Also, they didn't expose the CameraManager, which is needed to get at the cam list. It's supposed to be a singleton and this was leading to crashes. Sure I could have posted a PR there and let them handle it, but for now my focus has been on getting things to work in the first place. And finally this thing crashed left and right for all sorts of reasons and it was my hope to get at least some context info by having the source included. A probably better way would be to strip out the parts we need and do things directly, not using libcamera-apps, but that again would be a "copy", albeit a possibly buggy one. So copying the source was the most economic choice IMO. |
If you want you can start the workflows, but don't merge just yet. There's issues with the framerates I hopefully get to work today or tomorrow. |
@anjok How is it going? I see a lot of progress in the INDI forum thread. |
@anjok is this driver usable at this stage? |
@knro it didn't crash in a while, but I didn't get around to do as much testing as I'd wish. And it's still weird in that raw pics don't take up changed settings unless you change them in jpg mode first. Also, since it includes the libcamera apps code, you need to pin it to specific versions of boost and whatever else it needs. But I guess it'd be the same with the shared apps libs. It takes take pics, most of the settings work and you can "use" it though. |
Ok, perhaps we can merge it now so we can get more feedback from users? |
In your place, I'd probably build and test it locally, but sure, go ahead. |
No description provided.