kayg/calternal
Watch
1
0
Fork
You've already forked calternal
 0
Code Issues 191 Pull requests 61 Projects Releases Packages Wiki Activity Actions 20

Opacity pixel checks wait for settled WebKit frames (#442) #524

Closed
kayg wants to merge 0 commits from issue/442-opacity-flake into dev
pull from: issue/442-opacity-flake
merge into: kayg:dev
Conversation 1 Commits - Files changed -
kayg commented 2026-10-17 06:53:16 +00:00
Owner
Copy link

References #442.

The opacity test waited a fixed 700 ms after the remote photo became shown. That signal starts a 450 ms CSS fade, but a wall clock delay does not guarantee WebKit has painted the settled frame under load. The test now waits for CSS motion to finish and for two frames before sampling the 100% and 0% endpoints. The 60% and 20% alpha checks remain. Pixel decoding now uses the existing page instead of opening a temporary browser page.

The color thresholds are unchanged.

Before repro, webkit-1440 --repeat-each=10:

Error: blue shows through at 0% (full)
Expected: > 63
Received: 39
6 failed
64 passed (20.8m)

After targeted WebKit run, 30 repeats:

30 passed (9.6m)

Four project matrix, --repeat-each=3:

18 skipped
66 passed (11.2m)

bun run check / svelte-check:

COMPLETED 526 FILES 0 ERRORS 12 WARNINGS 7 FILES_WITH_PROBLEMS

The 12 warnings are in existing files outside this change.

References #442. The opacity test waited a fixed 700 ms after the remote photo became shown. That signal starts a 450 ms CSS fade, but a wall clock delay does not guarantee WebKit has painted the settled frame under load. The test now waits for CSS motion to finish and for two frames before sampling the 100% and 0% endpoints. The 60% and 20% alpha checks remain. Pixel decoding now uses the existing page instead of opening a temporary browser page. The color thresholds are unchanged. Before repro, `webkit-1440 --repeat-each=10`: ```text Error: blue shows through at 0% (full) Expected: > 63 Received: 39 6 failed 64 passed (20.8m) ``` After targeted WebKit run, 30 repeats: ```text 30 passed (9.6m) ``` Four project matrix, `--repeat-each=3`: ```text 18 skipped 66 passed (11.2m) ``` `bun run check` / svelte-check: ```text COMPLETED 526 FILES 0 ERRORS 12 WARNINGS 7 FILES_WITH_PROBLEMS ``` The 12 warnings are in existing files outside this change.
E2E: sample opacity after backdrop settles (#442)
Some checks failed
CI / server (pull_request) Successful in 31s
Details
CI / rust (pull_request) Failing after 2m38s
Details
CI / web (pull_request) Failing after 41m44s
Details
a2e841cce9
kayg commented 2026-10-17 12:57:44 +00:00
Author
Owner
Copy link

Landed in train w2, #532.

Landed in train w2, #532.
kayg closed this pull request 2026-10-17 12:57:45 +00:00
Some checks failed
CI / server (pull_request) Successful in 31s
Details
CI / rust (pull_request) Failing after 2m38s
Details
CI / web (pull_request) Failing after 41m44s
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
backend
Rust server

  
design
Mockups, UI and TASTE

  
docs
ADRs, CONTEXT, TASTE

  
idea
Parked idea, never worked until promoted

  
infra
Deploy, CI, VMs

  
perf
Performance and budgets

  
reliability
Data integrity, corruption traps

  
web
Web client

No labels
backend
design
docs
idea
infra
perf
reliability
web
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
kayg/calternal!524
No description provided.