Skip to content

Release/3.1 - merge 406: Don't display invalid results#479

Merged
camillobruni merged 1 commit into
WebKit:release/3.1from
camillobruni:2025-01-29_backmerge_invalid_results
Feb 18, 2025
Merged

Release/3.1 - merge 406: Don't display invalid results#479
camillobruni merged 1 commit into
WebKit:release/3.1from
camillobruni:2025-01-29_backmerge_invalid_results

Conversation

@camillobruni
Copy link
Copy Markdown
Contributor

@camillobruni camillobruni commented Jan 29, 2025

This partially fixes #399 by not display non-finite scores which are caused by 0-measured suite results.

  • For non-positive or non-finite scores, the following summary page is displayed. Note that the detailed page is still accessible since you might find some useful information there.
  • Additionally, a console.error is generated for each zero-sum suite

[backmerge of #406]

This partially fixes WebKit#399 by not display non-finite scores which are caused by 0-measured suite results.

- For non-positive or non-finite scores, the following summary page is displayed. Note that the detailed page is still accessible since you might find some useful information there.
- Additionally, a console.error is generated for each zero-sum suite
@camillobruni camillobruni changed the base branch from main to release/3.1 January 29, 2025 15:33
@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 29, 2025

Deploy Preview for webkit-speedometer ready!

Name Link
🔨 Latest commit 550fc77
🔍 Latest deploy log https://app.netlify.com/sites/webkit-speedometer/deploys/679a4a5bfc77a500082db997
😎 Deploy Preview https://deploy-preview-479--webkit-speedometer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@camillobruni camillobruni changed the title Don't display invalid results (backmerge of #406 to v3.1) Release/3.1 - merge 406: Don't display invalid results Feb 11, 2025
@camillobruni camillobruni merged commit ce03cfb into WebKit:release/3.1 Feb 18, 2025
@camillobruni camillobruni deleted the 2025-01-29_backmerge_invalid_results branch February 18, 2025 15:31
camillobruni added a commit to camillobruni/Speedometer that referenced this pull request Jun 2, 2026
b475f2d Perf-Dashboard.Render sync phase sets location.hash, but the actual DOM (WebKit#488) (WebKit#496)
b270564 (origin/3.1, 3.1) Release/3.1 428 - add id to input (NEW - with correct base to merge into) (WebKit#494)
aba1d5e (thor/release/3.1, flashd/release/3.1) Deconfuse Geomean Metric and geomean aggregate value (WebKit#408) (WebKit#480)
ce03cfb Don't display invalid results (WebKit#406) (WebKit#479)
6f25345 Update Version Text from 3.0 to 3.1 (WebKit#485)
cc9ee08  Release/3.1 - merge 400 (rename className -> class) (WebKit#483)
e12e84c Release/3.1 - merge 448 (news site article ids nuxt) (WebKit#482)
8a4a566 Release/3.1 - merge 447 (news site article ids next) (WebKit#481)

Change-Id: If0b19391128b82389eda1a61d33dc4e2be2cd180
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

With privacy.resistFingerprinting set to true in Firefox, Speedometer shows the result Infinity

3 participants