Skip to content

Update Speedometer2.1 content to latest runner - #348

Merged
camillobruni merged 146 commits into
WebKit:speedometer2.1-contentfrom
camillobruni:speedometer2.1-content
Jan 30, 2024
Merged

Update Speedometer2.1 content to latest runner#348
camillobruni merged 146 commits into
WebKit:speedometer2.1-contentfrom
camillobruni:speedometer2.1-content

Conversation

@camillobruni

@camillobruni camillobruni commented Jan 16, 2024

Copy link
Copy Markdown
Contributor

Merge the speedometer2.1-content branch with the latest main branch to get the developer-menu.

flashdesignory and others added 30 commits May 2, 2023 08:13
* Add observable plot

* Add a grouped bar chart

* Add dot chart

* Add a reset button and minor refactoring

* Regenerate

* Cache mapped state for states

* Remove one intermediary array

* Comments tweaking

* Use vite machinery to inject the csv files as string in the production build

* Simplify the test

* Comment the Grouped Chart example

* Review comments: add more comments, extract some values to constants, and various other small things

* Add a README file

* Moved the developer page to developer.html and add a new index

* Remove the graph comments better, remove the unused parameter in the call to parseAirportsInformation and extract the number of airports to keep to a constant

* Remove addGroupedBars and related code

---------

Co-authored-by: Brian Grinstead <briangrinstead@gmail.com>
- Use more performance.mark for whole suites.
- Pre-build the mark strings upfront
We've noticed that we get lazy compilation events for test-runner methods.
By doing running a warmup suite first, we can avoid this addition noise during measurements.

- Add Warmup suite
- Add params.useWarmupSuite option
- Add better performance.measure annotation for all suites
- Enforce better default order of metrics
- Introduce Metric.separator to be used by metrics-ui and CSV generation
- Use Geomean instead of Total
WebKit#155)

* Make the number of airports per group configurable

* Add test steps to run drawing the graph with more items in each state
This allows the benchmark to be ran on a subdirectory.
…om. (WebKit#171)

Fixes WebKit#138
This allows the test runner to support an iteration count higher than 100.
For the upcoming comparison view we will need the separate data points which are not available in the existing JSON format.
- Add `Download classic JSON` button
- `Download JSON` and `Copy JSON` now provide a serialized version of the new metrics
- Make sure there are not loops in the metrics structure to JSON.stringify them directly
- Rename "Total" to  "Geomean"
This class abstracts away two nested setTimeout calls and requestAnimationFrame.
This PR adds a new query string parameter requestAnimationFrame, which takes
the value of either "timer" or "raf". "timer" is the existing and default
measurement method. When it's "raf", we use two consecutive requestAnimationFrame
first of which runs the sync step and second of which schedules a 0s timer to
signify the end of async step.
lpardosixtosMs and others added 24 commits August 30, 2023 10:44
This PR does not enable any test in the runner.

Co-authored-by: issack john <issackjohn@microsoft.com>
* Fix count
* Rewrite generator
* Wrap the generator in a function
* combine folderwrapper and treeitem
* add class for type of item
* Update dist folders

---------

Co-authored-by: Julien Wajsberg <felash@gmail.com>
Co-authored-by: Issack John <issack.john@outlook.com>
* Check prereqs in the build script and handle them
* add code comments
* Update build scripts & package.jsons
- Fix scroll content overflow
- Remove li spacing in the developer menu
waitOnElement uses a 50ms timeout, which was historically motivated by FlightJS that used requirejs which had a hard-coded 50ms pause.

We've observed fragile CPU scaling behavior with this approach in Speedometer 2.1. V3.0 is less affected by this since the perf-dashboard is the only workload the regularly needs another cycle to wait (all other workloads are ready after onload has fired).

Using rAF lowers the timeout limit and is more realistic in required CPU ramp-up performance.
…bKit#337)

Previously, we were transforming it into a set. This was giving problems
in a later condition, so that no message/debugInfo were defined, and the
user would get an alert with the message "undefined".

Now the set stays local to the code block where it's used, so that the
parameter variable "tags" is an array for the whole function, for more
consistency with the "names" variable.
…ic WebKit pathology (WebKit#338)

* Utility function to create css variants for big-dom.css
* update build scripts
* update dists
@camillobruni
camillobruni changed the base branch from main to speedometer2.1-content January 16, 2024 12:18
@camillobruni
camillobruni requested a review from rniwa January 16, 2024 12:23
@rniwa rniwa added the trivial change A change that doesn't affect benchmark results label Jan 27, 2024
@camillobruni
camillobruni merged commit 491acc2 into WebKit:speedometer2.1-content Jan 30, 2024
@camillobruni
camillobruni deleted the speedometer2.1-content branch January 30, 2024 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

trivial change A change that doesn't affect benchmark results

Projects

None yet

Development

Successfully merging this pull request may close these issues.