HDR extension doesn't work on Samsung Galaxy A52s 5G [364152642]

Fixed

Bug

Status Update

No update yet.

Description

ch...@google.com

created issue #1

Sep 3, 2024 05:50AM

This issue is created for tracking the forum issue link.

When the HDR extension on the CameraX library is used (while using the back camera), while the preview is running logcat displays these lines continuously:

2024-08-30 09:58:54.550 21414-21461 cx/CaptureCallback      com.mypackage             V  onCaptureProcessStarted, seq = 1
2024-08-30 09:58:54.555 21414-21461 cx/CaptureCallback      com.mypackage             V  onCaptureCompleted, seq = 1, timestamp = 66168716018710
2024-08-30 09:58:54.592 21414-21460 cx/CaptureCallback      com.mypackage             V  onCaptureStarted, seq = 1, timestamp = 66168776051314
2024-08-30 09:58:54.610 21414-21453 cx/CaptureCallback      com.mypackage             V  onCaptureProcessStarted, seq = 1
2024-08-30 09:58:54.616 21414-21453 cx/CaptureCallback      com.mypackage             V  onCaptureCompleted, seq = 1, timestamp = 66168776051314
2024-08-30 09:58:54.652 21414-21459 cx/CaptureCallback      com.mypackage             V  onCaptureStarted, seq = 1, timestamp = 66168836083971
2024-08-30 09:58:54.672 21414-21460 cx/CaptureCallback      com.mypackage             V  onCaptureProcessStarted, seq = 1
2024-08-30 09:58:54.679 21414-21460 cx/CaptureCallback      com.mypackage             V  onCaptureCompleted, seq = 1, timestamp = 66168836083971
2024-08-30 09:58:54.713 21414-21453 cx/CaptureCallback      com.mypackage             V  onCaptureStarted, seq = 1, timestamp = 66168896116575
2024-08-30 09:58:54.733 21414-21461 cx/CaptureCallback      com.mypackage             V  onCaptureProcessStarted, seq = 1
2024-08-30 09:58:54.739 21414-21461 cx/CaptureCallback      com.mypackage             V  onCaptureCompleted, seq = 1, timestamp = 66168896116575
2024-08-30 09:58:54.773 21414-21460 cx/CaptureCallback      com.mypackage             V  onCaptureStarted, seq = 1, timestamp = 66168956149179
2024-08-30 09:58:54.795 21414-21459 cx/CaptureCallback      com.mypackage             V  onCaptureProcessStarted, seq = 1
2024-08-30 09:58:54.800 21414-21459 cx/CaptureCallback      com.mypackage             V  onCaptureCompleted, seq = 1, timestamp = 661689561491

This goes on and on. When capturing a photo, the capture callback is never called. In logcat I can see the following errors:

Surface no longer in use[total_surfaces=6, used_surfaces=4](androidx.camera.core.impl.SessionProcessorSurface@1258a6c}
--
CameraDevice.onError(): 0 failed with ERROR_CAMERA_DEVICE while in OPENED state. Will attempt recovering from error.
--
Attempt to reopen camera[0] after error[ERROR_CAMERA_DEVICE]
--
Session 0: Exception while stopping repeating: android.hardware.camera2.CameraAccessException: CAMERA_ERROR (3): The camera device has encountered a serious error at android.hardware.camera2.impl.CameraDeviceImpl.checkIfCameraClosedOrInError(CameraDeviceImpl.java:2513)

See the attached logcat file log1.txt.

When using the front camera, the preview issues the same logs while the preview is running. Capturing a photo sometimes work (looks like it works when there is not a lot of difference in light intensity). When it does not work, the following error is logged:

Tag name 'dynamicShotCaptureDuration' does not exist.

See the attached logcat file log2.txt.

I followed the API documentation. The sample library (https://github.com/android/camera-samples) has the same behaviour. I also tried a third party CameraX app from the Play Store, taking a photo in HDR causes the same issue. Using code or an app that implements Camera2 directly does not cause any problems with HDR enabled.

Occurs with CameraX version: 1.3.4 and 1.4.0-rc01 I tried both PreviewView.ImplementationMode.PERFORMANCE and PreviewView.ImplementationMode.COMPATIBLE Extensions Bokeh and Face Retouch do work.

Device: Samsung Galaxy A52s 5G

Model: SM-A528B/DS

Android version: 14 (build number UP1A.231005.007.A528BXXS9GXG5)

log1.txt

46 KB

View

Download

log2.txt

6.3 KB

View

Download

Comments

ch...@google.com <ch...@google.com> Sep 3, 2024 09:42AM

Accepted by ch...@google.com.

ch...@google.com <ch...@google.com> #2Sep 3, 2024 12:10PM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit c60f33e229e31ab328ef6b59dab63b264954831c
Author: Alexandre Elias <aelias@google.com>
Date: Fri Jul 10 16:23:09 2020

Semantics no-op cleanups

Partly in response to lmr's broad code review, I did a pass of
superficial API/implementation cleanup. The main changes are:

- I changed each Boolean SemanticsProperty where false is equivalent to
not being present to take "Unit" instead. This is conceptually
clearer: it avoids questions like "can I cancel out a semantics from a
merged child by setting it to false?" Because "property = Unit" looks
weird, I also changed the style of these to "property()".

- I moved the Semantics id generator closer to where it's used, in
SemanticsModifierCore. I made it internal and an AtomicInt.
(Note that integer ids are heavily used in the Android
AccessibilityNodeInfo APIs so I can't simply remove them entirely.)

- I deleted dead code. Some examples include SemanticsHintOverrides,
a public API not connected to anything, and SemanticsPropertyKey
merge() open method which is never called. (In both cases I have
a different plan in mind for accessibility.)

Fixes: 145951226
Fixes: 145955412
Test: existing tests
Relnote: "Single-value semantics properties now use a calling style.
For example, 'semantics { hidden = true }' is now written as:
'semantics { hidden() }'."

Change-Id: Ic1afd12ea22c926babc9662f1804d80b33aa0cfc

M ui/integration-tests/benchmark/src/androidTest/java/androidx/ui/benchmark/test/LayoutNodeModifierBenchmark.kt
M ui/ui-core/api/0.1.0-dev15.txt
M ui/ui-core/api/current.txt
M ui/ui-core/api/public_plus_experimental_0.1.0-dev15.txt
M ui/ui-core/api/public_plus_experimental_current.txt
M ui/ui-core/api/restricted_0.1.0-dev15.txt
M ui/ui-core/api/restricted_current.txt
M ui/ui-core/src/androidAndroidTest/kotlin/androidx/ui/graphics/vector/VectorTest.kt
M ui/ui-core/src/androidAndroidTest/kotlin/androidx/ui/semantics/SemanticsTests.kt
M ui/ui-core/src/androidMain/kotlin/androidx/ui/core/AndroidActuals.kt
M ui/ui-core/src/androidMain/kotlin/androidx/ui/core/AndroidComposeView.kt
M ui/ui-core/src/androidMain/kotlin/androidx/ui/core/AndroidComposeViewAccessibilityDelegateCompat.kt
M ui/ui-core/src/androidMain/kotlin/androidx/ui/core/AndroidPopup.kt
M ui/ui-core/src/commonMain/kotlin/androidx/ui/core/Expect.kt
M ui/ui-core/src/commonMain/kotlin/androidx/ui/core/semantics/SemanticsConfiguration.kt
D ui/ui-core/src/commonMain/kotlin/androidx/ui/core/semantics/SemanticsHintOverrides.kt
M ui/ui-core/src/commonMain/kotlin/androidx/ui/core/semantics/SemanticsModifier.kt
M ui/ui-core/src/commonMain/kotlin/androidx/ui/core/semantics/SemanticsNode.kt
M ui/ui-core/src/commonMain/kotlin/androidx/ui/core/semantics/SemanticsOwner.kt
M ui/ui-core/src/commonMain/kotlin/androidx/ui/core/semantics/SemanticsWrapper.kt
M ui/ui-core/src/commonMain/kotlin/androidx/ui/semantics/SemanticsProperties.kt
M ui/ui-foundation/api/0.1.0-dev15.txt
M ui/ui-foundation/api/current.txt
M ui/ui-foundation/api/public_plus_experimental_0.1.0-dev15.txt
M ui/ui-foundation/api/public_plus_experimental_current.txt
M ui/ui-foundation/api/restricted_0.1.0-dev15.txt
M ui/ui-foundation/api/restricted_current.txt
M ui/ui-foundation/src/main/java/androidx/ui/foundation/Clickable.kt
M ui/ui-foundation/src/main/java/androidx/ui/foundation/Dialog.kt
M ui/ui-foundation/src/main/java/androidx/ui/foundation/Scroller.kt
M ui/ui-foundation/src/main/java/androidx/ui/foundation/selection/Selectable.kt
M ui/ui-foundation/src/main/java/androidx/ui/foundation/selection/Toggleable.kt
M ui/ui-foundation/src/main/java/androidx/ui/foundation/semantics/FoundationSemanticsProperties.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/ButtonTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/CardTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/CheckboxScreenshotTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/RadioButtonScreenshotTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/ScaffoldTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/SnackbarTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/SurfaceTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/ripple/RippleIndicationTest.kt
M ui/ui-material/src/androidTest/java/androidx/ui/material/textfield/TextFieldScreenshotTest.kt
M ui/ui-material/src/main/java/androidx/ui/material/AppBar.kt
M ui/ui-material/src/main/java/androidx/ui/material/TextFieldImpl.kt
M ui/ui-test/src/androidTest/java/androidx/ui/test/AssertsTest.kt
M ui/ui-test/src/androidTest/java/androidx/ui/test/CallSemanticsActionTest.kt
M ui/ui-test/src/androidTest/java/androidx/ui/test/ErrorMessagesTest.kt
M ui/ui-test/src/androidTest/java/androidx/ui/test/FindersTest.kt
M ui/ui-test/src/androidTest/java/androidx/ui/test/PrintToStringTest.kt
M ui/ui-test/src/androidTest/java/androidx/ui/test/ScrollToTest.kt
M ui/ui-test/src/androidTest/java/androidx/ui/test/TextActionsTest.kt
M ui/ui-test/src/main/java/androidx/ui/test/Actions.kt
M ui/ui-test/src/main/java/androidx/ui/test/Filters.kt
M ui/ui-text/api/0.1.0-dev15.txt
M ui/ui-text/api/current.txt
M ui/ui-text/api/public_plus_experimental_0.1.0-dev15.txt
M ui/ui-text/api/public_plus_experimental_current.txt
M ui/ui-text/api/restricted_0.1.0-dev15.txt
M ui/ui-text/api/restricted_current.txt
M ui/ui-text/src/commonMain/kotlin/androidx/ui/text/CoreTextField.kt
M ui/ui-text/src/commonMain/kotlin/androidx/ui/text/TextSemanticsProperties.kt

https://android-review.googlesource.com/1360099

AllOkSwInfo-A52s5G-UP1A.231005.007.A528BXXS6FXA2.png

188 KB

View

Download

AllNgSwInfo-A52s5G-UP1A.231005.007.A528BXXU9GXH1.png

178 KB

View

Download

dumpstate-2024-09-03-02-49-38.zip

20 MB

Download

ap...@google.com <ap...@google.com> #3Sep 6, 2024 06:24AM

Project: platform/frameworks/support
Branch: androidx-main

commit 06a8cbbd9a1922c51361ab59155ae2003fa98c67
Author: CharcoalChen <charcoalchen@google.com>
Date: Thu Sep 05 14:02:23 2024

Disable extensions for Samsung A52s' back camera which id is 0

Even if the extensions ability of camera id 0 is disabled, the Samsung A52s 5G device still has the other camera with id 2 can support extensions and work well for the back lens facing. Applications can still take pictures with extensions mode on the back len facing camera if they do not specify the specific camera id 0 to get the extensions enabled camera selector.

Relnote: "Disabled extensions for Samsung A52s' back camera which id is 0, because native crashes might happen when capturing HDR images and configuring capture sessions might fail for the BOKEH or FACE_RETOUCH modes."

Bug: 364152642
Test: ./gradlew bOS
Change-Id: I03ec9ce16ff100c38eb9286cb391c4e32ee5c728

M camera/camera-extensions/src/androidTest/java/androidx/camera/extensions/ExtensionsManagerTest.kt
M camera/camera-extensions/src/androidTest/java/androidx/camera/extensions/util/ExtensionsTestUtil.java
M camera/camera-extensions/src/main/java/androidx/camera/extensions/internal/AdvancedVendorExtender.java
M camera/camera-extensions/src/main/java/androidx/camera/extensions/internal/BasicVendorExtender.java
M camera/camera-extensions/src/main/java/androidx/camera/extensions/internal/compat/quirk/ExtensionDisabledQuirk.java
M camera/camera-extensions/src/main/java/androidx/camera/extensions/internal/compat/workaround/ExtensionDisabledValidator.java
M camera/camera-extensions/src/test/java/androidx/camera/extensions/internal/compat/workaround/ExtensionDisabledValidatorTest.kt

https://android-review.googlesource.com/3256459

ch...@google.com <ch...@google.com> #4Sep 9, 2024 05:42AM

Marked as fixed.

Hi,

https://android-review.googlesource.com/3256459 has been merged to workaround the problem. The fix will be included in CameraX's next 1.5.0 alpha release. We will also discuss to cherry-pick the solution to have a 1.4.0-rc release for this problem.

Before getting the CameraX releases with the fix, you can consider the following approaches to workaround this issue in your app:

Force disable CameraX extensions for the back camera (id 0) on the Samsung A52s devices in your app.
Force select camera id 2 for back lens facing setting with extensions enabled on the Samsung A52s devices in your app.

Once we get any update from Samsung, I will update the obtained info to this issue.

pr...@google.com <pr...@google.com> #5Oct 2, 2024 05:37PM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.camera:camera-extensions:1.5.0-alpha02

Issue 364152642

Description

Issue summary

Comments

ch...@google.com <ch...@google.com> Sep 3, 2024 09:42AM

ch...@google.com <ch...@google.com> #2Sep 3, 2024 12:10PM

ap...@google.com <ap...@google.com> #3Sep 6, 2024 06:24AM

ch...@google.com <ch...@google.com> #4Sep 9, 2024 05:42AM

pr...@google.com <pr...@google.com> #5Oct 2, 2024 05:37PM

Add comment

Issue metadata