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
[ONNX] Remove protocol dataclass #85916
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/85916
Note: Links to docs will display an error until the docs builds have been completed. ❌ 2 FailuresAs of commit 69cfdf9: The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reference to issue of dataclasses Inheriting from Protocol python/cpython#89244
@pytorchbot merge -g |
@pytorchbot successfully started a merge job. Check the current status here. |
Merge failedReason: The following mandatory check(s) failed (Rule Dig deeper by viewing the failures on hud Details for Dev Infra teamRaised by workflow job |
@pytorchbot rebase |
@pytorchbot successfully started a rebase job. Check the current status here |
- Remove the `_WithOp` protocol because it is not used and causes the dataclass GraphContext to not be able to init in some python versions
Successfully rebased |
d9c52c5
to
69cfdf9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd highly recommend making changes to the CI to make sure this limitation would have caught by the pipelines before merging it
@pytorchbot merge -f "Flacky doc test and cuda tests" |
@pytorchbot successfully started a merge job. Check the current status here. |
Remove the `_WithOp` protocol because it is not used and causes the dataclass `GraphContext` to not be able to init in some python versions. Reference to issue of dataclasses Inheriting from Protocol python/cpython#89244 Pull Request resolved: #85916 Approved by: https://github.com/BowenBao, https://github.com/abock, https://github.com/thiagocrepaldi
Remove the `_WithOp` protocol because it is not used and causes the dataclass `GraphContext` to not be able to init in some python versions. Reference to issue of dataclasses Inheriting from Protocol python/cpython#89244 Pull Request resolved: pytorch#85916 Approved by: https://github.com/BowenBao, https://github.com/abock, https://github.com/thiagocrepaldi
Remove the
_WithOp
protocol because it is not used and causes the dataclassGraphContext
to not be able to init in some python versions.Reference to issue of dataclasses Inheriting from Protocol python/cpython#89244