CameraX Extensions - Renaming "Beauty" to "Face Retouch" [198515274]

Fixed

Feature Request

Status Update

No update yet.

Description

er...@google.com

created issue #1

Sep 2, 2021 12:57AM

This is a request to rename the CameraX Extension "Beauty" to "Face Retouch".

Beauty implies a subjective standard
The actual algorithm applies retouching to which this is more accurately named

Comments

gf...@google.com <gf...@google.com> #2Sep 2, 2021 01:18AM

d.a.c page: might need to change the wording on that page,

https://developer.android.com/training/camerax/vendor-extensions

sc...@google.com <sc...@google.com> Sep 2, 2021 03:17AM

Reassigned to ch...@google.com.

ap...@google.com <ap...@google.com> #3Sep 3, 2021 01:24AM

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

commit b23be7f7903c4350f2846a71961ed517cded2f47
Author: Charcoal Chen <charcoalchen@google.com>
Date: Thu Sep 02 21:09:22 2021

Rename ExtensionMode#BEAUTY to FACE_RETOUCH

Relnote: "Renamed ExtensionMode#BEAUTY to FACE_RETOUCH to correctly present what is done by the extension mode."

Bug: 198515274
Test: ./gradlew camera:camera-extensions:build && ./gradlew camera:camera-extensions:connectedAndroidTest
Change-Id: I61f544ce058d1ee39722825fd95e7bb1e1ac6349

M camera/camera-extensions/src/androidTest/java/androidx/camera/extensions/util/ExtensionsTestUtil.java
M camera/camera-extensions/src/main/java/androidx/camera/extensions/ExtensionMode.java
M camera/camera-extensions/src/main/java/androidx/camera/extensions/ExtensionsInfo.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/integration-tests/extensionstestapp/src/androidTest/java/androidx/camera/integration/extensions/PreviewProcessorTimestampTest.java
M camera/integration-tests/extensionstestapp/src/main/java/androidx/camera/integration/extensions/CameraExtensionsActivity.java

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

ch...@google.com <ch...@google.com> #4Sep 3, 2021 02:03AM

"Beauty" term still exists on DAC in two places:

Overview page

These effects include Portrait, HDR, Night, and Beauty.

The first set of capabilities available include Portrait, HDR, Night, and Beauty. These capabilities are available on supported devices.

Vendor extensions page

BeautyImageCaptureExtender
BeautyPreviewExtender

For the Overview page, we can rename those words directly. For the Vendor extensions page, we can sync the Legacy Extensions API Removal paragraph in CameraX Extensions API (Draft) to avoid mentioning the legacy class name.

er...@google.com <er...@google.com> #5Sep 3, 2021 05:04PM

Adding @gfan - Hi Gerry - can you help with comment#4? Would you prefer another bug?

er...@google.com <er...@google.com> Sep 3, 2021 06:26PM

Reassigned to gf...@google.com.

gf...@google.com <gf...@google.com> Sep 3, 2021 08:52PM

Accepted by gf...@google.com.

ch...@google.com <ch...@google.com> #6Sep 3, 2021 10:55PM

For the comment#4 Overview page part, cl/394594779 has been uploaded and got LGTM from dactyl. I'll merge it later.

gf...@google.com <gf...@google.com> #7Sep 3, 2021 11:34PM

Oh, created cl/394772017, it does not conflict with yours. So feel free to merge yours. Thank you

gf...@google.com <gf...@google.com> #8Sep 8, 2021 10:01PM

Marked as fixed.

cl/395565590 submitted. this could be closed now.