Repository navigation
Recover live updates and validate integrated performance, visuals and network attribution - #39
Merged
Merged
Conversation
…network attribution
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
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. Comment |
Tree 394fdb4 is unchanged from the head verified by Windows run 38207323090 and paired study 38207323082. This commit resolves overlapping merge ancestry after PR37 without changing source files.
This was referenced Oct 12, 2026
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
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.
Why
A transient sampling exception could permanently stop live updates, and an earlier successful snapshot could still let an unhealthy UI benchmark pass. Failed service refreshes need an accessible stale-data indication, and Performance selection needs consistent behavior when a device disappears.
Changes
Verified evidence
Windows verification passed all 12 CTests, 354 named native self-test checks, all 21 suite cases, Python checks on Windows/Linux, production exclusions, and packaging. All 18 retained UI reports meet the existing responsiveness budgets; the worst queue p95 was 32.2983 ms against 100 ms. These reports were checked offline without rerunning benchmarks.
The paired study completed 18 serial AB/BA/AB measurements. An independent audit matched 486 metric values to raw reports and recomputed all 81 summary rows. Live Windows UDP passed with 32 datagrams, 32,768 bytes, and 64 own-process events; no skip was used.
All 38 native light/dark views were inspected. Missing GPU-engine feedback on the VM is being corrected in a focused follow-up. These images do not establish physical GPU or complete Windows 11 coverage.
The main timing ranges overlap. Favorable sampler/idle CPU medians do not establish a general speedup. Production grew by 22,016 bytes; diagnostic private memory and working set were somewhat higher. Full results and limits will be preserved in the release performance report.
Integration
After #37 merged, overlapping ancestry required a merge-resolution commit. Its tree remains exactly
394fdb43ac86e09a2d30eba5eb4e399d4ef3d50e, identical to the fully verified candidate. No source file changed. The one-off performance-study label was removed after reviewing the completed evidence to avoid measuring unchanged contents again. Normal CI still runs for the integration commit.