Spec: Power Display ambient-light (ALS) adaptive brightness for external monitors#49199
Open
MrRishabhJain wants to merge 2 commits into
Open
Spec: Power Display ambient-light (ALS) adaptive brightness for external monitors#49199MrRishabhJain wants to merge 2 commits into
MrRishabhJain wants to merge 2 commits into
Conversation
…xternal monitors Design spec for a per-monitor adaptive brightness mode in Power Display driven by the device ambient light sensor, bridging Light Switch's ALS reader and Power Display's DDC/CI brightness control. Tracking issue: microsoft#49038. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ust lifecycle Incorporates the design discussed on microsoft#49038: unified per-monitor adaptive-curve + offset model (answers configurability, CLI/manual +/-, and slider behaviour), per-monitor calibration rationale for multi-monitor sync, and a dedicated Sensor trust & lifecycle section (lid-closed detection, hold-last-good, clamshell boundary, fallbacks). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Design spec for ambient-light (ALS) adaptive brightness for external monitors in Power Display, as discussed and agreed in #49038.
The feature continuously maps the device''s ambient light sensor reading (the same ALS Light Switch already reads) to each external monitor''s brightness over DDC/CI (the path Power Display already uses) — i.e. auto-brightness for external monitors, the continuous/sensor-driven counterpart to the schedule-based #47480.
This PR adds only the spec doc (
doc/specs/power-display-adaptive-brightness.md), per the repo''s spec-first process (doc/specs/readme.md). No code.What the spec covers
target = clamp(curve(lux) + offset, min, max).pausebehaviour.GUID_LIDSWITCH_STATE_CHANGE+QueryDisplayConfig), hold-last-good instead of dimming to black, and honest fallbacks.Discussion / sign-off
Behaviour was reviewed with @moooyo on #49038 (configurability, CLI/manual +/- while adaptive, slider behaviour) — agreed to proceed.
Closes nothing yet; tracks #49038. Related: #47480, #42566, #35564.