Skip to content

Conversation

lukaszstolarczuk
Copy link
Contributor

@lukaszstolarczuk lukaszstolarczuk commented Mar 12, 2025

  • remove the second, test commit

@lukaszstolarczuk
Copy link
Contributor Author

lukaszstolarczuk commented Mar 12, 2025

@ianayl , @pbalcer - I guess this branch should be rebased with latest sycl - what's your preferred approach for this...?

EDIT: I forgot, the version of python scripts and the code source is taken from sycl branch. Once #17443 is merged I can test it again here.

@lukaszstolarczuk lukaszstolarczuk marked this pull request as ready for review March 13, 2025 14:26
@lukaszstolarczuk lukaszstolarczuk requested a review from a team as a code owner March 13, 2025 14:26
@lukaszstolarczuk
Copy link
Contributor Author

@sarnex, thx for approve, I cancelled the CI as I didn't think it's neccessary to run on this branch - I only tested here if the benchmarks works fine. Should we merge this now or should we restart the CI?

@sarnex
Copy link
Contributor

sarnex commented Mar 13, 2025

@lukaszstolarczuk Can you remind me when the CI that this change effects runs? Is it the nightly?

@lukaszstolarczuk
Copy link
Contributor Author

@lukaszstolarczuk Can you remind me when the CI that this change effects runs? Is it the nightly?

If you're asking about ur-benchmarks-reusable.yml, then it's only called on demand via ur-benchmarks.yml at the moment.

@sarnex
Copy link
Contributor

sarnex commented Mar 13, 2025

got it, thanks

@sarnex sarnex merged commit 6620e4a into intel:unify-benchmark-ci Mar 13, 2025
0 of 2 checks passed
@lukaszstolarczuk lukaszstolarczuk deleted the bump-compute-bench branch March 14, 2025 09: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.

2 participants