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

TakePhotoActivity: Use full field of view #2016

Merged
merged 1 commit into from
May 12, 2022

Conversation

ASerbinski
Copy link
Contributor

@ASerbinski ASerbinski commented May 10, 2022

Some devices have a digital video stabilization mode that uses a smaller field of view within the full set of sensor data, and moves
the selected subset around like on rubber bands in order to reduce shaking in video captures. Most devices default this mode to OFF, but some, like Pixel 6 Pro, default this mode to ON, which causes image and video captures to have a reduced field of view.

This change will have no effect on most devices, but will increase field of view to full on those devices that do not by default.

Reference: https://issuetracker.google.com/issues/230013960

Signed-off-by: Adam Serbinski [email protected]

Some devices have a digital video stabilization mode that uses a
smaller field of view within the full set of sensor data, and moves
the selected subset around like on rubber bands in order to reduce
shaking in video captures. Most devices default this mode to OFF,
but some, like Pixel 6 Pro, default this mode to ON, which causes
image and video captures to have a reduced field of view.

This change will have no effect on most devices, but will increase
field of view to full on those devices that do not by default.

Reference: https://issuetracker.google.com/issues/230013960

Signed-off-by: Adam Serbinski <[email protected]>
@mahibi
Copy link
Collaborator

mahibi commented May 12, 2022

so i guess merge without being able to test.. as you implemented the camera i requested a review @AndyScherzinger

1 similar comment
@mahibi
Copy link
Collaborator

mahibi commented May 12, 2022

so i guess merge without being able to test.. as you implemented the camera i requested a review @AndyScherzinger

@AndyScherzinger
Copy link
Member

@ASerbinski btw we added you to the nextcloud org back then, see https://github.com/orgs/nextcloud/people?query=ASerbinski so you can actually push to this repo directly to a branch, so you don't have to maintain a fork of the repo. makes things easier, just wanted to let you know in case you forgot :)

@AndyScherzinger
Copy link
Member

Code-wise looks good to me 👍 So merging!

Thanks again for this contribution @ASerbinski all your work around the camera implementation is highly appreciated - makes my day 🙏

@AndyScherzinger AndyScherzinger merged commit ffaa638 into nextcloud:master May 12, 2022
@AndyScherzinger AndyScherzinger added this to the 14.1.0 milestone May 12, 2022
@mahibi
Copy link
Collaborator

mahibi commented May 13, 2022

thank you @ASerbinski 👍 👍

@ASerbinski
Copy link
Contributor Author

I'm happy to contribute any time I have something to offer.

@AndyScherzinger I actually forgot about that. I'll keep it in mind for next time. I plan to try to add a toggle for EIS on video calls along the same lines as I contributed to grapheneos camera; GrapheneOS/Camera@4eafcbb

@AndyScherzinger
Copy link
Member

@ASerbinski that is so awesome! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants