Skip to content

Conversation

ianayl
Copy link
Contributor

@ianayl ianayl commented Mar 13, 2025

TODO: still figuring out the best way to do this

ianayl and others added 30 commits February 27, 2025 14:01
This patch improves numerous aspects on how the benchmarking
results are visualized:
 - rewrites the way HTML charts are generated, using a library (Chart.js)
 that's both easier to use and more visually pleasing.
 The new HTML page also now decouples data from the HTML itself,
 leading to faster load times and the ability to fetch data
 from remote sources.
 - The markdown output now contains a failures section that
 lists all benchmarks that failed for a given run. This will be
 a helpful for developers during PR testing.
 - Benchmarks can now have description that's displayed on the page.
 - And many more minor improvements.
On PRs based on main, the scripts location is "old" and not accesible.
Pick location based on the dir existance. Step 'gather info' is in
a 'weird' location, so solve it with 2 tries to execute the script.
@ianayl ianayl temporarily deployed to WindowsCILock March 13, 2025 16:44 — with GitHub Actions Inactive
@ianayl ianayl temporarily deployed to WindowsCILock March 13, 2025 16:44 — with GitHub Actions Inactive
@ianayl ianayl changed the title [CI] Enable UR benchmarking dashboard in intel/llvm [CI][Do not merge] Enable UR benchmarking dashboard in intel/llvm Mar 13, 2025
@ianayl ianayl temporarily deployed to WindowsCILock March 13, 2025 17:50 — with GitHub Actions Inactive
@ianayl ianayl temporarily deployed to WindowsCILock March 13, 2025 17:50 — with GitHub Actions Inactive
@ianayl ianayl temporarily deployed to WindowsCILock March 13, 2025 18:21 — with GitHub Actions Inactive
@ianayl
Copy link
Contributor Author

ianayl commented May 1, 2025

Already merged in another PR

@ianayl ianayl closed this May 1, 2025
@ianayl ianayl deleted the ianayl/split-benchmark-ci branch May 1, 2025 15:38
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.

3 participants