Skip to content

[SYCL] Fix postcommit again #14385

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

Merged
merged 1 commit into from
Jul 2, 2024
Merged

[SYCL] Fix postcommit again #14385

merged 1 commit into from
Jul 2, 2024

Conversation

sarnex
Copy link
Contributor

@sarnex sarnex commented Jul 2, 2024

Unused param/func warnings

Signed-off-by: Sarnie, Nick <[email protected]>
@sarnex sarnex temporarily deployed to WindowsCILock July 2, 2024 14:45 — with GitHub Actions Inactive
@sarnex sarnex marked this pull request as ready for review July 2, 2024 14:52
@sarnex sarnex requested a review from a team as a code owner July 2, 2024 14:52
@sarnex sarnex requested a review from againull July 2, 2024 14:52
@sarnex
Copy link
Contributor Author

sarnex commented Jul 2, 2024

FYI @KseniyaTikhomirova

@sarnex sarnex merged commit bdeb3ce into intel:sycl Jul 2, 2024
5 of 6 checks passed
@KseniyaTikhomirova
Copy link
Contributor

FYI @KseniyaTikhomirova

sorry and thanks a lot!

@sarnex
Copy link
Contributor Author

sarnex commented Jul 2, 2024

No problem, precommit passed so it's not your fault

@KseniyaTikhomirova
Copy link
Contributor

I see TEST 'SYCL :: Basic/stream/blocking_pipes_and_stream.cpp' FAILED, working on this.

@sarnex
Copy link
Contributor Author

sarnex commented Jul 2, 2024

Thanks!

@sarnex
Copy link
Contributor Author

sarnex commented Jul 2, 2024

@KseniyaTikhomirova BTW if you can't fix the test immediately please submit a PR disabling it for now to unblock others. thanks!

@KseniyaTikhomirova
Copy link
Contributor

@KseniyaTikhomirova BTW if you can't fix the test immediately please submit a PR disabling it for now to unblock others. thanks!

It fails with timeout so I would suggest disable it for now #14392
I will work on fix and reenabling with top priority

@KseniyaTikhomirova
Copy link
Contributor

more fixes #14396

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