Skip to content
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] Improve error handling for adaptive_pool #45874

Closed
wants to merge 17 commits into from

Conversation

neginraoof
Copy link
Contributor

Duplicate of #43032
This update would also improve error handling for interpolate with 'area' mode.

Copy link
Collaborator

@BowenBao BowenBao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resubmitting, LGTM

@codecov
Copy link

codecov bot commented Oct 6, 2020

Codecov Report

Merging #45874 into master will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #45874      +/-   ##
==========================================
- Coverage   68.32%   68.31%   -0.01%     
==========================================
  Files         410      410              
  Lines       52992    52997       +5     
==========================================
- Hits        36208    36207       -1     
- Misses      16784    16790       +6     
Impacted Files Coverage Δ
torch/onnx/symbolic_opset9.py 35.39% <0.00%> (-0.18%) ⬇️
torch/onnx/utils.py 70.29% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4ab73c1...0d12708. Read the comment docs.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bzinodev has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@neginraoof
Copy link
Contributor Author

@bzinodev Can we merge this PR please? Thanks!

@facebook-github-bot
Copy link
Contributor

@bzinodev merged this pull request in 5ce31b6.

neginraoof added a commit to neginraoof/pytorch that referenced this pull request Oct 9, 2020
Summary:
Duplicate of pytorch#43032
This update would also improve error handling for interpolate with 'area' mode.

Pull Request resolved: pytorch#45874

Reviewed By: albanD

Differential Revision: D24141266

Pulled By: bzinodev

fbshipit-source-id: 7559f1d6af4f1ef3507c15a1aee76fe01fa433cd
malfet pushed a commit that referenced this pull request Oct 12, 2020
Summary:
Duplicate of #43032
This update would also improve error handling for interpolate with 'area' mode.

Pull Request resolved: #45874

Reviewed By: albanD

Differential Revision: D24141266

Pulled By: bzinodev

fbshipit-source-id: 7559f1d6af4f1ef3507c15a1aee76fe01fa433cd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants