Camera icon indicating copy to clipboard operation
Camera copied to clipboard

EIS not fully working on Pixel 6

Open Silent-Hunter opened this issue 2 years ago • 13 comments

EIS works fine when not recording. I can tilt the phone slightly and the image in the viewfinder is stable. But as soon as I start recording, it's as if the EIS is off. It's been like this for over a year.

GrapheneOS build number is UP1A.231005.007.2023102300

Graphene Camera is version 63

Silent-Hunter avatar Oct 26 '23 16:10 Silent-Hunter

It seems to be intentional

Gets disabled before recording starts here https://github.com/GrapheneOS/Camera/blob/53efbad43aaaeb3a8996c6c77da9d84b2de22dda/app/src/main/java/app/grapheneos/camera/capturer/VideoCapturer.kt#L288

Gets re-enabled when recording stops here https://github.com/GrapheneOS/Camera/blob/53efbad43aaaeb3a8996c6c77da9d84b2de22dda/app/src/main/java/app/grapheneos/camera/capturer/VideoCapturer.kt#L326

soupslurpr avatar Nov 02 '23 22:11 soupslurpr

What on Earth? Why? That seems odd.

Silent-Hunter avatar Nov 03 '23 00:11 Silent-Hunter

@soupslurpr That code prevents changing the state of the EIS toggle after starting the video. It doesn't disable EIS.

thestinger avatar Nov 03 '23 04:11 thestinger

Ah, sorry for misunderstanding that. Is it a bug then that EIS doesn't seem to work when starting the recording? I noticed this too on my Pixel 7

soupslurpr avatar Nov 03 '23 04:11 soupslurpr

I can confirm that reports look valid. Move the EIS switch to "on" and the field of view narrows to provide the feature's required buffer, and shaking the phone around a bit, the image appears to be stabilized. As soon as recording is enabled though, the video no longer appears to be stabilized, but still has the narrowed field of view.

My first thought is that it could be connected to some kind of logic that may disable EIS if the camera has OIS capability. Of note, Pixel 6 does have OIS on the main lens, but if I remember correctly, not on the wide angle or telephoto (pro model) lenses.

ASerbinski avatar Apr 22 '24 22:04 ASerbinski

Hey everyone,

We have filed an issue on CameraX's issue tracker for the same. The status of the filed issue can be tracked from here - https://issuetracker.google.com/issues/346692558.

Thanks a lot for reporting this issue!

MHShetty avatar Jun 12 '24 13:06 MHShetty

Hey everyone,

Could someone please help us out with testing a fix for this issue?

MHShetty avatar Jun 17 '24 15:06 MHShetty

How would one do that?

Silent-Hunter avatar Jun 19 '24 16:06 Silent-Hunter

How would one do that?

Hey @Silent-Hunter, are you present on Element?

MHShetty avatar Jun 19 '24 22:06 MHShetty

I can test too :)

soupslurpr avatar Jun 20 '24 05:06 soupslurpr

Hey @Silent-Hunter, are you present on Element?

No, sorry.

Silent-Hunter avatar Jun 20 '24 07:06 Silent-Hunter

Hey @Silent-Hunter, are you present on Element?

No, sorry.

Ohh no worries @Silent-Hunter. Thanks for reaching out!

MHShetty avatar Jun 20 '24 08:06 MHShetty

Hey everyone,

We have tried using the official CameraX API for video stabilization on a Pixel 7 device and that doesn't seem to work with the preview as well currently unfortunately. We have reported this issue on the official CameraX Issue Tracker.

Further updates on this issue can be tracked on https://issuetracker.google.com/issues/346692558 until there's any update directly mentioned on this GitHub issue.

Special thanks to @soupslurpr for helping us out with testing with their Pixel 7!

MHShetty avatar Jun 21 '24 05:06 MHShetty

Hey everyone,

Does EIS work for anyone who was facing issues previously, in the latest release?

MHShetty avatar Aug 03 '24 15:08 MHShetty

Its working perfectly for me, much smoother with it on vs off, good job!

soupslurpr avatar Aug 03 '24 16:08 soupslurpr

Its working perfectly for me, much smoother with it on vs off, good job!

Perfect, glad to know @soupslurpr. Thanks for confirming this!

MHShetty avatar Aug 03 '24 18:08 MHShetty

@Silent-Hunter @ASerbinski Could you please confirm if the issue is fixed on your end in the latest release? We can raise the issue further to the CameraX team just in case if the issue still persists. Thanks a lot for your time and help!

MHShetty avatar Aug 05 '24 21:08 MHShetty

Is 73 the latest release? That's the version I currently have; I'm not sure how to update it beyond that manually, if it's not.

EDIT: It does seem to be working! It feel less pronounced than when it used to only work in the preview, but it does seem to be doing something.

Silent-Hunter avatar Aug 05 '24 22:08 Silent-Hunter

Is 73 the latest release? That's the version I currently have; I'm not sure how to update it beyond that manually, if it's not.

Yes 73 is the latest version and it should have the changes for EIS included in it

EDIT: It does seem to be working! It feel less pronounced than when it used to only work in the preview, but it does seem to be doing something.

Ohh glad to know that! Is there any difference between the expected behavior for EIS, and what you're experiencing in the preview and the video that gets recorded?

MHShetty avatar Aug 06 '24 06:08 MHShetty

I also feel like its less pronounced but its actually a good thing IMO. Before it kind of locked on to a specific spot and didn't allow movement until you moved it a lot and then it again locked on which was annoying but now it allows motion while keeping it smooth. Its a great improvement. The locked mode could be useful I suppose but I definitely think the way it is now makes more sense at least as a default.

soupslurpr avatar Aug 06 '24 06:08 soupslurpr

What soupslurpr said, basically! It's less pronounced, but the recording now matches the preview, and while the more aggressive mode could definitely be useful for certain things, it's not a necessity.

Silent-Hunter avatar Aug 06 '24 06:08 Silent-Hunter

We can have a separate issue about adding more configuration. It will depend on what CameraX offers now.

thestinger avatar Aug 06 '24 07:08 thestinger

@Silent-Hunter @soupslurpr Glad to know that this issue has been fixed. Thanks a lot for reporting and confirming this issue!

MHShetty avatar Aug 06 '24 10:08 MHShetty