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] ONNX dev branch merge 01-06-2021 #50163
Conversation
💊 CI failures summary and remediationsAs of commit 795c9e1 (more details on the Dr. CI page):
ci.pytorch.org: 1 failedThis comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.Please report bugs/suggestions to the (internal) Dr. CI Users group. This comment has been revised 11 times. |
Codecov Report
@@ Coverage Diff @@
## master #50163 +/- ##
==========================================
- Coverage 80.68% 80.64% -0.05%
==========================================
Files 1904 1906 +2
Lines 206555 207383 +828
==========================================
+ Hits 166667 167250 +583
- Misses 39888 40133 +245 |
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.
@SplitInfinity has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Just a heads up - unless this stack is created with |
Hi @SplitInfinity - Thanks for the heads up. Is there no way to capture the commits separately? At the very least, can we put a commit message which captures the titles of main commits as sub-bullets (manually entered)? So, a message like this: [ONNX] ONNX dev branch merge 01-06-2021
|
Yes. Edit the contents of your first comment in this thread to include such a list, and that's what will be included in the commit message by our tooling. |
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.
@SplitInfinity has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
#50237 is attempting to make a similar change to one of the commits in this stack. If that lands first (which it likely will since it's smaller), you'll have to rebase. |
Thanks @SplitInfinity. Now that #50237 has been merged, I will update this PR. |
* [ONNX] Add checks in ONNXSetDynamicInputShape * [ONNX] Add checks in ONNXSetDynamicInputShape
* Add derive_index * Add derive_index test * Adding more tests * Update symbolic_opset9.py
* update symbolic for unfold * update symbolic_opse12 file * update symbolic_opse12 file * [ONNX] Support onnx if/loop sequence output in opset 13 - (pytorch#49270) * Symbolic function for torch.square (pytorch#49446) * instead of a pass use a helper function * update ort version * Revert "instead of a pass use a helper function" This reverts commit 723b446. * update symbolics * update symbolic * update symbolics * [ONNX] Support onnx if/loop sequence output in opset 13 - (pytorch#49270) * Symbolic function for torch.square (pytorch#49446) * empty commit * fix clang-tidy * fix clang-tidy Co-authored-by: Bowen Bao <bowbao@microsoft.com> Co-authored-by: David Fan <30608893+jiafatom@users.noreply.github.com>
…it is as same as inputs. (pytorch#49798) * [ONNX] Support onnx if/loop sequence output in opset 13 - (pytorch#49270) * Symbolic function for torch.square (pytorch#49446) * [ONNX] Support onnx if/loop sequence output in opset 13 - (pytorch#49270) * Symbolic function for torch.square (pytorch#49446) * Update code so that initializers' sequence is as same as inputs. * Correct the format according to flake8. * Correct the format by clang-format. * Add a new test for script model. * Update expect files for Test_Operators tests. Co-authored-by: Bowen Bao <bowbao@microsoft.com> Co-authored-by: David Fan <30608893+jiafatom@users.noreply.github.com>
* Enable opset 13 ORT tests * Update test.sh * Set environ var * Update test.sh * Enabling more ops for opset 13 * change master to main * Update symbolic_opset13.py * Flake 8 fix * [ONNX] Support onnx if/loop sequence output in opset 13 - (pytorch#49270) * Symbolic function for torch.square (pytorch#49446) * Clean up tests * Exclude more tests * Trigge build * [ONNX] Support onnx if/loop sequence output in opset 13 - (pytorch#49270) * Symbolic function for torch.square (pytorch#49446) * update ORT version * disable more tests * clean up * flake8 * Disable TV tests * Update test_pytorch_onnx_onnxruntime.py Co-authored-by: Bowen Bao <bowbao@microsoft.com> Co-authored-by: David Fan <30608893+jiafatom@users.noreply.github.com>
* add pass * add tests * enable tests * update dtype symbolic * update pass * clang format * update utils file * fix clang_tidy errors * update pass * update pass * clang format * update pass * update pass * update pass * update pass * update pass * update pass * update pass * update pass * fix mypy tests * update comment * update pass * add new line * update pass * empty commit * empty commit * fix merge conflict
33f26ed
to
7b920b0
Compare
Rebased and reconciled with latest master, including #50237. |
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.
@SplitInfinity has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@spandantiwari |
Thanks @SplitInfinity, we are taking a look. |
|
@SplitInfinity - We have fixed the test failure. In the latest CI, there's a test failing, but it seems unrelated to this PR. In fact we found a couple of other PRs which are seeing the same failure (#50155, #50157), which probably points towards some unrelated issue. |
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.
@SplitInfinity has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Yeah, those tests look unrelated. I imported the diff again, I'll let you know how the internal tests look. |
Thanks @SplitInfinity . :-) |
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.
@SplitInfinity has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@SplitInfinity merged this pull request in aeefe2c. |
[ONNX] ONNX dev branch merge 01-06-2021