Skip to content

[plugin][benchmark] upload OOT benchmark results to OOB dashboard#501

Open
gbyu-amd wants to merge 3 commits intomainfrom
guanbao/benchmark_to_oob_dashboard
Open

[plugin][benchmark] upload OOT benchmark results to OOB dashboard#501
gbyu-amd wants to merge 3 commits intomainfrom
guanbao/benchmark_to_oob_dashboard

Conversation

@gbyu-amd
Copy link
Copy Markdown
Contributor

@gbyu-amd gbyu-amd commented Apr 7, 2026

Motivation

This PR enhanced the OOT benchmark workflow to automatically upload the benchmark results to OOB dashboard as well.
Since OOB dashboard requires the benchmark results to be uploaded as an excel file with some specific format, a new script was added to process the data:

  • .github/scripts/oot_benchmark_to_oob_dashboard.py

Similarly, the OOT benchmark workflow was updated to make the uploading optional when running the benchmark:
image

Since OOB data upload requires a password, a new repo secret, namely OOB_BENCHMARK_DATA_UPLOAD_PW, has been added in the settings.
image

If benchmark finished and upload succeed, a screenshot will be saved to indicate the status. The following image comes with this run: https://github.com/ROCm/ATOM/actions/runs/24074200879, where gpt-oss and kimi k2 were selected as an example.
oob_dashboard_upload
The data shown in OOB dashboard is aligned with the ones from benchmark run.

Test Plan

Manually trigger a benchmark run to verify the functionality.

Test Result

The benchmark data has been successfully uploaded as expected.

Submission Checklist

@gbyu-amd gbyu-amd marked this pull request as draft April 7, 2026 07:08
@gbyu-amd gbyu-amd requested a review from zejunchen-zejun April 7, 2026 07:08
@gbyu-amd gbyu-amd marked this pull request as ready for review April 7, 2026 14:22
@gbyu-amd gbyu-amd requested review from valarLip and wuhuikx April 7, 2026 14:22
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