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.

Sign up for GitHub

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

Jump to bottom

Use torch.testing.assert_close in test_video_reader.py #3888

Merged
merged 25 commits into from
May 21, 2021
Merged

Use torch.testing.assert_close in test_video_reader.py #3888

merged 25 commits into from
May 21, 2021

Conversation

NicolasHug
Copy link
Member

part of #3865

pmeier and others added 23 commits May 20, 2021 14:09
@pmeier
adopt torch.testing.assert_close in test suite
c4fc01b
@pmeier
revert some changes
bfbe19b
@pmeier
add todo
09f86f4
@pmeier
flake8
86402f0
@NicolasHug
Hopefully fixed test_functional_tensor
48d32e6
@NicolasHug
hopefully fixed test_ops
15b50e3
@NicolasHug
Merge branch 'master' of github.com:pytorch/vision into assert-close
a54880f
@NicolasHug
Fix test_utils
61874ac
@NicolasHug
revert unwanted changes to test_image
30f20a3
@NicolasHug
maybe fixed test_transforms
3a29ae3
@NicolasHug
Merge branch 'master' into assert-close
d6d73d0
@pmeier
fix test_datasets_video_utils
863f144
@pmeier
fix test_transforms
c8a5afa
@pmeier
Merge branch 'master' into assert-close
e697e88
@pmeier
flake8
93614f0
@NicolasHug
Merge branch 'master' of github.com:pytorch/vision into assert-close
11caf01
@NicolasHug
Merge branch 'assert-close' of github.com:pmeier/vision into assert-c…
d7fde8c 
…lose
@NicolasHug
use cu102 see if the nightlies are actual nightlies?
0b237c7
@NicolasHug
obviously forgot to call regenerate.py
c2ace86
@NicolasHug
not as obvious, reverting
d78226a
@NicolasHug
Merge branch 'master' into assert-close
bb543a7
@NicolasHug
Merge branch 'master' into assert-close
7507a0c
@NicolasHug
revert everything but test_video_reader
89cf705
NicolasHug
NicolasHug commented May 21, 2021
View reviewed changes
Copy link
Member Author

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

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

LGTM!

datumbox
datumbox approved these changes May 21, 2021
View reviewed changes
Copy link
Contributor

@datumbox datumbox left a comment

Choose a reason for hiding this comment

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

Stamping

@NicolasHug NicolasHug merged commit 74559c4 into pytorch:master May 21, 2021
facebook-github-bot pushed a commit that referenced this pull request May 25, 2021
@datumbox @facebook-github-bot
[fbsync] Use torch.testing.assert_close in test_video_reader.py (#3888)
471ea8b 
Summary: Co-authored-by: Philip Meier <github.pmeier@posteo.de>

Reviewed By: vincentqb, cpuhrsch

Differential Revision: D28679985

fbshipit-source-id: 849b5b9076fb2238d4d3b026e2afc7152f539575
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@datumbox datumbox datumbox approved these changes

Assignees
No one assigned
Labels
cla signed code quality module: tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@NicolasHug @datumbox @facebook-github-bot @pmeier