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_transforms_video.py #3886

Merged
merged 25 commits into from
May 21, 2021
Merged

Use torch.testing.assert_close in test_transforms_video.py #3886

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 transforms_video
636f282
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 6fa9308 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_transforms_video.py (#…
a71b537 
…3886)

Summary: Co-authored-by: Philip Meier <github.pmeier@posteo.de>

Reviewed By: vincentqb, cpuhrsch

Differential Revision: D28679968

fbshipit-source-id: cc92cfa4eb77ad740f671a49c4a4aa0d92764230
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