Skip to content

[SYCL] [Graph] Fix print_graph #19181

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 2 commits into from
Jun 27, 2025
Merged

[SYCL] [Graph] Fix print_graph #19181

merged 2 commits into from
Jun 27, 2025

Conversation

keyradical
Copy link
Contributor

Fixes #19177

@keyradical keyradical requested a review from a team as a code owner June 26, 2025 16:30
@keyradical keyradical requested a review from fabiomestre June 26, 2025 16:30
- Correct extra underscore in preview changes guard for print_graph
- Also restore WIN32 code for export of print_graph as a test
@Bensuo
Copy link
Contributor

Bensuo commented Jun 27, 2025

I've pushed some changes that should resolve the issues that were in CI for this PR as well as the original issue.

@sarnex I understand that some of the CI runners are down right now, though I have tested these changes locally on windows and linux for the E2E and ABI tests, as well as for the pre-C++11 ABI on linux so I'm pretty confident this should fix the reported issues. Would you prefer to wait till next week to try and merge this once CI is all back up, or merge it sooner to try and fix the post-commit?

I'd guess maybe the post-commit is also affected by CI problems so merging it now might not be worth it but not sure.

@sarnex
Copy link
Contributor

sarnex commented Jun 27, 2025

Yeah like half of the machines are down due to some lab maintenance until Monday. Given the scope of the change I'll trust you :)

@sarnex sarnex merged commit 3740e5a into intel:sycl Jun 27, 2025
19 of 22 checks passed
@Bensuo Bensuo temporarily deployed to WindowsCILock June 27, 2025 14:33 — with GitHub Actions Inactive
@Bensuo Bensuo temporarily deployed to WindowsCILock June 27, 2025 17:34 — with GitHub Actions Inactive
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.

Graph tests failing in nightly
3 participants