[ONNX] Skip empty input test case in aten_mm#118413
[ONNX] Skip empty input test case in aten_mm#118413titaiwangms wants to merge 2 commits intopytorch:mainfrom
Conversation
titaiwangms
requested review from
BowenBao,
abock,
thiagocrepaldi and
wschin
as code owners
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/118413
Note: Links to docs will display an error until the docs builds have been completed. ❌ 1 New FailureAs of commit 962acee with merge base 2842d3c ( NEW FAILURE - The following job has failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
titaiwangms
added
module: onnx
|
@pytorchbot merge |
|
@titaiwangms the merge will fail because of a broken CI check: Check mergeability of ghstack PR / ghstack-mergeability-check (pull_request) Failing after 3s We have to wait for all CI checks are green and do a merge -f afterwards |
Merge startedYour change will be merged once all checks pass (ETA 0-4 Hours). Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Merge failedReason: 1 jobs have failed, first few of them are: Check mergeability and dependencies for ghstack prs / ghstack-mergeability-check Details for Dev Infra teamRaised by workflow job |
|
@pytorchbot merge -f "Check mergeability of ghstack PR / ghstack-mergeability-check (pull_request) is flaky" |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
Thanks! |
Fixes pytorch#117718 Fixes pytorch#117725 It's actually a known issue in microsoft/onnxscript#586, and we do exclude the empty input test cases in aten_matmul. This PR follows the skip, and add aten_mm as well. Pull Request resolved: pytorch#118413 Approved by: https://github.com/thiagocrepaldi
4 participants
Fixes #117718
Fixes #117725
It's actually a known issue in microsoft/onnxscript#586, and we do exclude the empty input test cases in aten_matmul. This PR follows the skip, and add aten_mm as well.