Skip to content

Add seeds on Kernel Info and reduce randomness for Gaussian Blur #6741

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

Merged
merged 3 commits into from
Oct 11, 2022

Conversation

datumbox
Copy link
Contributor

Trying to reduce the flakiness on the prototype gaussian_blur kernel. This PR:

  • Adds an optional seed in all KernelInfo objects that can be set at the beginning of each test prior generating the data.
  • Removes the random sampling from the sizes/frames from Gaussian Blur data because these random calls are not covered by the seed.

@datumbox datumbox requested a review from pmeier October 11, 2022 13:49
@datumbox datumbox marked this pull request as draft October 11, 2022 13:49
@datumbox datumbox changed the title [WIP] Add seeds on Kernel Info and reduce randomness for Gaussian Blur Add seeds on Kernel Info and reduce randomness for Gaussian Blur Oct 11, 2022
@datumbox datumbox marked this pull request as ready for review October 11, 2022 15:24
@datumbox datumbox requested a review from vfdev-5 October 11, 2022 15:24
@vfdev-5
Copy link
Collaborator

vfdev-5 commented Oct 11, 2022

Thanks @datumbox , I rerun ubuntu test one more time to check.

@datumbox
Copy link
Contributor Author

@vfdev-5 Yeah good call. I couldn't make the test fail locally at all. Tried every seed in [0,100] and also randomly sampled but nothing. I'm not 100% this resolves the issue, let's see what the rerun does.

@datumbox
Copy link
Contributor Author

@vfdev-5 It seems it passed again. I'm going to merge because this flaky test is driving me nuts.

@pmeier If I messed up something or you favour a different approach, feel free to revert or update my approach.

@datumbox datumbox merged commit 6e72f2f into pytorch:main Oct 11, 2022
@datumbox datumbox deleted the tests/guassian_blur branch October 11, 2022 16:00
pmeier pushed a commit to pmeier/vision that referenced this pull request Oct 12, 2022
pmeier added a commit that referenced this pull request Oct 12, 2022
* Revert "Add seeds on Kernel Info and reduce randomness for Gaussian Blur (#6741)"

This reverts commit 6e72f2f.

* add fixture to fix the RNG seed

* re-add changes to gaussian_blur_* sample input shapes

Co-authored-by: Vasilis Vryniotis <[email protected]>
facebook-github-bot pushed a commit that referenced this pull request Oct 17, 2022
…Blur (#6741)

Summary:
* Add seeds on Kernel Info and reduce randomness for Gaussian Blur

* Fix linter

Reviewed By: NicolasHug

Differential Revision: D40427486

fbshipit-source-id: 89c2310929084115d0537df369ec1a525c842d43
facebook-github-bot pushed a commit that referenced this pull request Oct 17, 2022
Summary:
* Revert "Add seeds on Kernel Info and reduce randomness for Gaussian Blur (#6741)"

This reverts commit 6e72f2f.

* add fixture to fix the RNG seed

* re-add changes to gaussian_blur_* sample input shapes

Reviewed By: NicolasHug

Differential Revision: D40427476

fbshipit-source-id: b0aae476915f2d4e58146edf8e9339338bd78ad5

Co-authored-by: Vasilis Vryniotis <[email protected]>
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.

3 participants