-
Notifications
You must be signed in to change notification settings - Fork 17
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
Camera2 / CameraX API #32
Comments
We need to use Camera2 for sure, even if it's more complicated than CameraX. I think it's not possible to do a refactoring with the existing code, there are to many changes which are not compatible. I currently think the solution would be to refactor the main screen, and to allow to select between the current Interface and the new Interface. @woheller69 what are your thoughts about it? |
I think that is a good idea. Simply migrating will probably break a lot of things. And I currently have no time for such a major rewrite. If someone is willing to invest time and we can do it step by step without breaking existing code I am fine with this. |
Thanks! I work on this, as I still have an issue with crashing camera. |
This is my current progress. It's not finished yet, but it's all compiling and nothing fully broken ;-) But the important part: If you switch the recording mode, the settings can be replaced, so the second view will be recoding dependent. @woheller69 what do you think about this? |
For the planning section (duration, star time, etc) there should also be a group. I would prefer to do it this way: |
And some day (in distant future) when everything works fine we might simple delete all the old code and only use the new API |
with "switch to the new settings implementation" I mean a new separate settingsFragment2, etc. |
The settings should be dependent on the recording mode. If you click to "Kamera Einstellungen" (Sorry, phone is set to German) you get the old settings without Stop settings, which are implemented different, and therefore could be easy split, REST API, which are new, and could be easy split.
Yes, but you know how long that this usually takes, because for this you need all features in the new implementation.
Already done: But this is not enough, because some startup code is also there, and this needs to moved. |
I thought about this, but didn't do it yet. |
I just compiled it. I have no issues to have the settings grouped. I think that is fine if also a planning section is added. Why do we need a new recording mode? There will still be jpg, mp4, time lapse mp4, right (maybe not all of them immediately).
and if this is checked the available (implemented) new recording modes and settings are shown. |
I could get a preview working with Camera 2, but it's just a sample, I need to implemented the whole configuration and at least a few new features (Focus, Camera Selection etc.) else it makes no sense to integrate it... I'm working on this |
I got today the first images with Camera 2 :-) There is still a lot to do, the most configuration is still missing. But I'm working on it. |
The Camera Preview is basically working.You can now here configure ISO and Exposure Time. So to do for now:
Camera Selection: Front / Back and, if available, select the available Lens (Macro, Wide angle). Focus will not be available in the Settings, because you cannot set it without preview. @woheller69 what do you think about? It's still in development, but if you like to test it, the screenshot is from camera2-mode branch on my Repo. |
Configuring camera in preview sounds good. |
This will be solved with #39 |
Merged into beta branch |
This #22 would need Camera2 API.
This #23 also.
Probably this #10 also.
I think this should be done. It can be done, without increasing the API Level, which is important for me, because I plan to use old Phones!
I'll probably have a look at it.
The text was updated successfully, but these errors were encountered: