-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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] Fix pad Circular Mode #86984
[ONNX] Fix pad Circular Mode #86984
Conversation
[ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/86984
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit 27aec03: This comment was automatically generated by Dr. CI and updates every 15 minutes. |
ghstack-source-id: b5bebd104febca5b69adf9761f00280737257b20 Pull Request resolved: #86984
@@ -1849,32 +1850,34 @@ def constant_pad_nd(g: jit_utils.GraphContext, input, padding, value): | |||
) | |||
|
|||
|
|||
@_onnx_symbolic("aten::_pad_circular") |
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 deleted this, as this function is actually a subfunction of "aten::pad"
@BowenBao PTAL |
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.
🚀
@pytorchbot rebase |
@pytorchbot successfully started a rebase job. Check the current status here |
In #73433, a ONNX test case is missed, and the result is incorrect when it is converted to ONNX. [ghstack-poisoned]
Successfully rebased |
ghstack-source-id: 4247085e98d5207665361af6789cd6a6819def2d Pull Request resolved: #86984
@pytorchbot merge -g |
Merge startedYour change will be merged once all checks on your PR pass since you used the green (-g) flag (ETA: 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
In pytorch#73433, a ONNX test case is missed, and the result is incorrect when it is converted to ONNX. Pull Request resolved: pytorch#86984 Approved by: https://github.com/BowenBao
In pytorch#73433, a ONNX test case is missed, and the result is incorrect when it is converted to ONNX. Pull Request resolved: pytorch#86984 Approved by: https://github.com/BowenBao
In pytorch#73433, a ONNX test case is missed, and the result is incorrect when it is converted to ONNX. Pull Request resolved: pytorch#86984 Approved by: https://github.com/BowenBao
Stack from ghstack (oldest at bottom):
In #73433, a ONNX test case is missed, and the result is incorrect when it is converted to ONNX.