Disable upsample optest#13135
Closed
zrphercule wants to merge 6 commits intopytorch:masterfrom
zrphercule:disable_upsample_optest
Closed
Disable upsample optest#13135zrphercule wants to merge 6 commits intopytorch:masterfrom zrphercule:disable_upsample_optest
zrphercule wants to merge 6 commits intopytorch:masterfrom
zrphercule:disable_upsample_optest
Conversation
Contributor
Author
|
@houseroad could you please give a stamp? Thanks! |
houseroad
reviewed
Oct 25, 2018
| # TODO(Rui Zhu): Enable upsample_op test by fixing the number of input. | ||
| # See https://github.com/onnx/onnx/pull/1467/files | ||
| # | ||
| # def test_upsample(self): |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
Member
|
To unblock onnx/onnx#1467 |
Contributor
Author
|
@houseroad done |
houseroad
reviewed
Oct 25, 2018
|
|
||
| class TestOperators(TestCase): | ||
|
|
||
| def skipAlwaysForCI(func): |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
houseroad
approved these changes
Oct 25, 2018
Member
houseroad
left a comment
There was a problem hiding this comment.
LG, if CI is green, just land it
Member
|
import? |
Contributor
Author
|
@houseroad sure. |
Contributor
facebook-github-bot
left a comment
There was a problem hiding this comment.
zrphercule has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Contributor
facebook-github-bot
left a comment
There was a problem hiding this comment.
houseroad is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
laurentdupin
pushed a commit
to laurentdupin/pytorch
that referenced
this pull request
Apr 24, 2026
Summary: Temporarily disable upsample tests. Pull Request resolved: pytorch#13135 Reviewed By: bddppq Differential Revision: D10859926 Pulled By: houseroad fbshipit-source-id: 9eb068198d43ba0939d81a9e41eb6f24ff19cb6d
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Temporarily disable upsample tests.