Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NFC][CI] Move UR benchmarking scripts to devops/scripts #17562

Merged
merged 2 commits into from
Mar 24, 2025

Conversation

ianayl
Copy link
Contributor

@ianayl ianayl commented Mar 20, 2025

Given that #17229 is massive, I am splitting the PR into smaller individual PRs such that reviewers would have an easier time. This PR prepares for changes in #17229 by moving the UR benchmarking scripts into devops.

For more context: A previous attempt at splitting is here #17545, but even this is way too big, and thus I'm splitting it into chunks.

This is a non-functional change, as these scripts not currently used anywhere right now... hopefully soon though!

@ianayl ianayl temporarily deployed to WindowsCILock March 20, 2025 23:31 — with GitHub Actions Inactive
@ianayl ianayl marked this pull request as ready for review March 20, 2025 23:33
@ianayl ianayl requested review from a team as code owners March 20, 2025 23:33
@ianayl ianayl temporarily deployed to WindowsCILock March 20, 2025 23:43 — with GitHub Actions Inactive
@ianayl ianayl temporarily deployed to WindowsCILock March 21, 2025 18:08 — with GitHub Actions Inactive
@ianayl
Copy link
Contributor Author

ianayl commented Mar 21, 2025

@intel/unified-runtime-reviewers @intel/llvm-reviewers-benchmarking Friendly ping, thanks!

@ianayl
Copy link
Contributor Author

ianayl commented Mar 24, 2025

@intel/unified-runtime-reviewers Friendly ping: this PR is unfortunately holding up more PRs from landing. Thanks in advance!

@ianayl
Copy link
Contributor Author

ianayl commented Mar 24, 2025

@intel/llvm-gatekeepers PR is ready for merge, thanks!

Test failure is a test that has been long abandoned, but has been triggered due to PR triggers still in place.

@martygrant martygrant merged commit 8acb53b into intel:sycl Mar 24, 2025
31 of 32 checks passed
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.

6 participants