Skip to content

Set seed of test_randomperspective_fill #4759

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 27, 2021

Conversation

NicolasHug
Copy link
Member

@NicolasHug NicolasHug commented Oct 27, 2021

Towards #4506

I'm just parametrizing over 10 seeds for this one. a83d683 shows that all CI jobs are happy even with 100 seeds.

cc @vfdev-5 @datumbox @pmeier

@facebook-github-bot
Copy link

facebook-github-bot commented Oct 27, 2021

💊 CI failures summary and remediations

As of commit a3076c8 (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@NicolasHug NicolasHug marked this pull request as ready for review October 27, 2021 13:47
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.

LGTM.

@NicolasHug NicolasHug merged commit 8a62d54 into pytorch:main Oct 27, 2021
facebook-github-bot pushed a commit that referenced this pull request Nov 1, 2021
Summary:
* setting 100 seeds, let's see if it fails

* Passed with 100, so setting 10

Reviewed By: datumbox

Differential Revision: D32064683

fbshipit-source-id: 7b52f5531cd0f95abd426df1e864cc6fb1a9c209
cyyever pushed a commit to cyyever/vision that referenced this pull request Nov 16, 2021
* setting 100 seeds, let's see if it fails

* Passed with 100, so setting 10
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