ImageAnalysis.setOutputImageRotationEnabled(true) doesn't work on Nexus 5 (Android 5) [240993561]

Fixed

Bug

Status Update

No update yet.

Description

li...@gmail.com

created issue #1

Aug 2, 2022 02:53AM

CAMERAX VERSION: 1.1.0 and 1.2.0-alpha04

DEVICE NAME: Nexus 5

DESCRIPTION: The image isn't rotated after calling setOutputImageRotationEnabled(true).

STEPS TO REPRODUCE:

Use the sample from https://github.com/android/camera-samples/tree/main/CameraXBasic
Open CameraFragment and debug the app. Record the image size in the ImageAnalyzer. It's 1280x720 in my case.
Set setOutputImageRotationEnabled(true).
Debug the app. The image size is supposed to be 720x1280, but it's still 1280x720.

Comments

wu...@google.com <wu...@google.com> #2Aug 3, 2022 12:36AM

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

commit 9905c64f5b25d9c0872472e0345c2474ff579f93
Author: Chris Craik <ccraik@google.com>
Date: Tue Mar 05 15:58:05 2024

Add newline to IDE warnings so it's on a separate line from results

Fixes: 328308833
Test: InstrumentationResultsTest
Relnote: "Fixes method tracing warning to be on separate line from
microbench output"

Change-Id: I0455c4edcdc592c56e60dd19bd32d9bd33f6af6a

M benchmark/benchmark-common/src/androidTest/java/androidx/benchmark/InstrumentationResultsTest.kt
M benchmark/benchmark-common/src/main/java/androidx/benchmark/InstrumentationResults.kt

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

wu...@google.com <wu...@google.com> Aug 3, 2022 09:02AM

Reassigned to ka...@google.com.

ka...@google.com <ka...@google.com> Aug 3, 2022 03:38PM

Assigned to ka...@google.com.

ka...@google.com <ka...@google.com> #3Aug 3, 2022 06:33PM

Accepted by ka...@google.com.

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

androidx.benchmark:benchmark-common:1.3.0-alpha02

ap...@google.com <ap...@google.com> #4Aug 25, 2022 12:02AM

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

commit cd926db1aaadb8f20f27cbc68cfbe7b73b879164
Author: Kailiang Chen <kailianc@google.com>
Date: Thu Aug 04 10:09:08 2022

[ImageAnalysis] Mark setOutputImageRotationEnabled as RequiresApi(23)

Relnote: "Add API level requirement for setOutputImageRotationEnabled"

Bug:

b/240993561

Test: Run integration test
Change-Id: I26e3e939c0d4662573840d378b802e68141b8283

M camera/camera-core/api/public_plus_experimental_current.txt
M camera/camera-core/src/main/java/androidx/camera/core/ImageAnalysis.java
M camera/camera-core/api/public_plus_experimental_1.2.0-beta02.txt
M camera/camera-core/api/restricted_current.txt
M camera/camera-core/api/1.2.0-beta02.txt
M camera/camera-core/api/current.txt
M camera/camera-core/api/restricted_1.2.0-beta02.txt

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

ka...@google.com <ka...@google.com> Aug 25, 2022 12:05AM

Marked as fixed.

na...@google.com <na...@google.com> #5Sep 21, 2022 05:54PM

This bug was linked in a change in the following release(s):

androidx.camera:camera-core:1.2.0-beta02