Skip to content

Conversation

NicolasHug
Copy link
Member

Summary:
The test is constantly failing: https://www.internalfb.com/intern/test/562949982577806?ref_report_id=0

The fix just adjusts atol from 1e-8 to 1e-7.

The equality test was likely failing on exact zeros

Reviewed By: fmassa

Differential Revision: D27790959

Summary:
The test is constantly failing: https://www.internalfb.com/intern/test/562949982577806?ref_report_id=0

The fix just adjusts `atol` from 1e-8 to 1e-7.

The equality test was likely failing on exact zeros

Reviewed By: fmassa

Differential Revision: D27790959

fbshipit-source-id: ed6fa7052b4f1cacc0dcbbbfd70d3554c05d02d9
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D27790959

@NicolasHug NicolasHug changed the title Fix torchvision_functional_tensor test_rgb2hsv [FBcode->GH] Fix torchvision_functional_tensor test_rgb2hsv Apr 15, 2021
facebook-github-bot pushed a commit that referenced this pull request Apr 15, 2021
Summary:
Pull Request resolved: #3676

The test is constantly failing: https://www.internalfb.com/intern/test/562949982577806?ref_report_id=0

The fix just adjusts `atol` from 1e-8 to 1e-7.

The equality test was likely failing on exact zeros

Reviewed By: fmassa

Differential Revision: D27790959

fbshipit-source-id: 58d06250df5905e39e197ee946ee2d875a5bab76
Copy link
Contributor

@prabhat00155 prabhat00155 left a comment

Choose a reason for hiding this comment

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

Thanks @NicolasHug for the PR.

@datumbox
Copy link
Contributor

Closing for the same reasons as #3675. If you intended to send a PR against master please reopen and change target branch.

@datumbox datumbox closed this Apr 17, 2021
NicolasHug added a commit to NicolasHug/vision that referenced this pull request Apr 19, 2021
Summary:
Pull Request resolved: pytorch#3676

The test is constantly failing: https://www.internalfb.com/intern/test/562949982577806?ref_report_id=0

The fix just adjusts `atol` from 1e-8 to 1e-7.

The equality test was likely failing on exact zeros

Reviewed By: fmassa

Differential Revision: D27790959

fbshipit-source-id: 58d06250df5905e39e197ee946ee2d875a5bab76
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.

4 participants