Skip to content

In pjrt runtime client, raise a Python exception if XLA compilation fails. #9138

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 10 commits into from
May 30, 2025

Conversation

zhanyong-wan
Copy link
Collaborator

@zhanyong-wan zhanyong-wan commented May 10, 2025

For #9096

Also remove some unnecessary #includes and deps.

@zhanyong-wan zhanyong-wan marked this pull request as draft May 10, 2025 00:25
@zhanyong-wan zhanyong-wan changed the title Throw a Python exception if compilation fails. In pjrt runtime client, raise a Python exception if XLA compilation fails. May 16, 2025
@zhanyong-wan zhanyong-wan marked this pull request as ready for review May 16, 2025 00:32
@zhanyong-wan zhanyong-wan requested a review from qihqi May 16, 2025 01:07
@zhanyong-wan zhanyong-wan marked this pull request as draft May 16, 2025 01:49
@zhanyong-wan zhanyong-wan removed the request for review from qihqi May 16, 2025 01:49
@zhanyong-wan zhanyong-wan marked this pull request as ready for review May 16, 2025 01:58
@zhanyong-wan zhanyong-wan requested a review from qihqi May 16, 2025 01:58
@zhanyong-wan zhanyong-wan marked this pull request as draft May 16, 2025 15:42
@zhanyong-wan zhanyong-wan force-pushed the crash2 branch 2 times, most recently from 456ba32 to 6f26279 Compare May 22, 2025 05:49
@zhanyong-wan zhanyong-wan requested a review from qihqi May 29, 2025 22:54
@zhanyong-wan zhanyong-wan marked this pull request as ready for review May 29, 2025 22:54
@zhanyong-wan zhanyong-wan requested a review from tengyifei May 30, 2025 01:51
@zhanyong-wan zhanyong-wan enabled auto-merge (squash) May 30, 2025 15:26
@zhanyong-wan zhanyong-wan merged commit d1ca3c4 into pytorch:master May 30, 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.

3 participants