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

Hi Ed, Thank you so much for these suggestions. I've been reviewing them and merging them in. Hopefully it should be live. I've included a thank you note too in the article.

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

Great! Thanks a lot, I'll look for the live updates soon!

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.