Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lighting correction #53

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

eehakkin
Copy link
Contributor

@eehakkin eehakkin commented Jan 25, 2022

Lighting correction corrects lighting balance between a foreground person and the background so that the foreground person is underexposed.

Please see #49 (comment) why a constrainable property is most suitable for the purpose.


Preview | Diff

@eehakkin eehakkin changed the title Lightning correction Lighting correction Mar 11, 2022
@eehakkin eehakkin force-pushed the feature/lighting-correction branch from ebaeeff to 70669ff Compare March 11, 2022 12:27
index.html Outdated Show resolved Hide resolved
@jan-ivar jan-ivar marked this pull request as draft October 13, 2022 14:57
@eehakkin eehakkin force-pushed the feature/lighting-correction branch 2 times, most recently from bf3fadf to b2854f0 Compare November 29, 2022 15:44
@eehakkin eehakkin marked this pull request as ready for review November 29, 2022 16:41
@eehakkin eehakkin force-pushed the feature/lighting-correction branch from a83a0cf to 5becfcc Compare March 29, 2023 07:56
@dontcallmedom-bot
Copy link

This issue was mentioned in WEBRTCWG-2023-02-21 (Page 46)

@aboba aboba added the TPAC 2023 label Sep 7, 2023
<p>When the "lightingCorrection" setting is set to <code>true</code> by
the <a>ApplyConstraints algorithm</a>, the UA will attempt to correct
human face and background lighting balance so that human faces are
not underexposed.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or overexposed?

@alvestrand
Copy link
Contributor

Probably should have a note saying that the meaning of "correct" is platform dependent and not controllable by the application.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants