Skip to content

Conversation

igchor
Copy link
Contributor

@igchor igchor commented May 22, 2025

To test OOO queues

@igchor igchor requested a review from a team as a code owner May 22, 2025 15:31
@igchor
Copy link
Contributor Author

igchor commented May 22, 2025

I'm not sure if we want all the scenarios that I added (SubmitKernel has a lot of variants already) but it would be nice to have comparison between SYCL, UR and L0

@pbalcer
Copy link
Contributor

pbalcer commented May 22, 2025

I'm not sure if we want all the scenarios that I added (SubmitKernel has a lot of variants already) but it would be nice to have comparison between SYCL, UR and L0

Ideally we'd wait until we deal with the long load times / run changes on the dashboard.... but yeah, we shouldn't worry too much about adding new scenarios.

@igchor igchor temporarily deployed to WindowsCILock May 22, 2025 16:10 — with GitHub Actions Inactive
@igchor igchor temporarily deployed to WindowsCILock May 22, 2025 17:19 — with GitHub Actions Inactive
@igchor igchor temporarily deployed to WindowsCILock May 22, 2025 17:19 — with GitHub Actions Inactive
@igchor igchor changed the title [Bench] add SubmitKernel benchmark variant wiht longer kernel [Bench] add SubmitKernel benchmark variant with longer kernel May 23, 2025
@igchor igchor temporarily deployed to WindowsCILock May 27, 2025 17:51 — with GitHub Actions Inactive
@igchor igchor temporarily deployed to WindowsCILock May 27, 2025 17:51 — with GitHub Actions Inactive
@igchor
Copy link
Contributor Author

igchor commented May 29, 2025

@intel/llvm-gatekeepers this is ready for merge, the failures are unrelated (this PR only modifies benchmark scripts)

@aelovikov-intel
Copy link
Contributor

@intel/llvm-gatekeepers this is ready for merge, the failures are unrelated (this PR only modifies benchmark scripts)

This will be the second PR where I ask folks working on benchmarks to fix CI not to run unnecessary tasks.

@PatKamin
Copy link
Contributor

@intel/llvm-gatekeepers this is ready for merge, the failures are unrelated (this PR only modifies benchmark scripts)

This will be the second PR where I ask folks working on benchmarks to fix CI not to run unnecessary tasks.

We'll mitigate this so to not to block the CI with bench PRs

@aelovikov-intel
Copy link
Contributor

@intel/llvm-gatekeepers this is ready for merge, the failures are unrelated (this PR only modifies benchmark scripts)

This will be the second PR where I ask folks working on benchmarks to fix CI not to run unnecessary tasks.

We'll mitigate this so to not to block the CI with bench PRs

Ian uploaded #18732

@aelovikov-intel aelovikov-intel merged commit 81a48a6 into intel:sycl May 29, 2025
24 of 25 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.

4 participants