Skip to content

fix(hdr): send active Windows display metadata for native HDR - #1048

Merged
capy-ai[bot] merged 1 commit into
devfrom
capy/monitor-aware-native-hdr
Sep 24, 2026
Merged

capy-ai[bot] merged 1 commit into
devfrom
capy/monitor-aware-native-hdr

Conversation

@zortos293

Copy link
Copy Markdown
Collaborator

Read the current Windows HDR output via DXGI and carry its measured minimum, peak, full-frame luminance, primaries, and white point through Qt’s transient runtime capability to CloudMatch. Match the window’s monitor, refresh on output changes, and omit incomplete metadata rather than treating SDR or stale values as calibration. An HDR-capable swapchain remains usable when measurements are missing.

Validate the optional monitor fields in the Rust core and serialize them in the official request units. Pair-only Wayland snapshots and the existing requested-content fallback remain unchanged. SDR still sends displayData:null; native HDR keeps the separate AI TrueHDR filter disabled. Document which mapping is inferred from the official client’s implementation rather than a captured HDR session.

Verification: core unit tests, cargo fmt --check, strict core Clippy, and git diff --check pass. Qt Windows tests and a live monitor comparison are pending on the local Windows build.

Open in Capy

@coderabbitai

coderabbitai Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: b5cf9fc3-de5d-43ae-9813-af8ec87ad4cd

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@zortos293
zortos293 marked this pull request as ready for review September 24, 2026 17:40
@capy-ai
capy-ai Bot merged commit 26dd41c into dev Sep 24, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant