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
OpInfos for torch.{flatten, column_stack} #67555
Conversation
[ghstack-poisoned]
CI Flow Status⚛️ CI FlowRuleset - Version:
You can add a comment to the PR and tag @pytorchbot with the following commands: # ciflow rerun, "ciflow/default" will always be added automatically
@pytorchbot ciflow rerun
# ciflow rerun with additional labels "-l <ciflow/label_name>", which is equivalent to adding these labels manually and trigger the rerun
@pytorchbot ciflow rerun -l ciflow/scheduled -l ciflow/slow For more information, please take a look at the CI Flow Wiki. |
🔗 Helpful links
💊 CI failures summary and remediationsAs of commit 8be8711 (more details on the Dr. CI page):
🕵️ 9 new failures recognized by patternsThe following CI failures do not appear to be due to upstream breakages: pytorch_linux_xenial_py3_6_gcc5_4_test (1/9)Step: "Test" (full log | diagnosis details | 🔁 rerun)
|
ghstack-source-id: 6c77b217d583090c9d1530ef4624fb8d19a0f86c Pull Request resolved: #67555
input_list = [] | ||
cases: Tuple[tuple, tuple] = ( # type: ignore[assignment] | ||
((S, 2, 1), (S, 3, 1)), | ||
((S), (S, 5)), ((), (1, S)) |
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.
((S), (S, 5))
-> ((S,), (S, 5))
?
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.
Awesome! Just a couple inline comments for your consideration.
Thanks @anjali411!
[ghstack-poisoned]
ghstack-source-id: 0b86f6443cbba7fca656578f7ebe427bf465ea8d Pull Request resolved: #67555
@anjali411 has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
@anjali411 merged this pull request in aceb46e. |
This pull request has been reverted by 14dc975. To re-land this change, follow these steps. |
Summary: Test Plan: Imported from OSS Reviewed By: cpuhrsch Differential Revision: D32650384 Pulled By: anjali411 fbshipit-source-id: 9ead83b378d0ece60569e1a0fc7d8849f89566b3 [ghstack-poisoned]
This pull request has been reverted by 14dc975. To re-land this change, follow these steps. |
Stack from ghstack:
Differential Revision: D32650384