Skip to content

Setting seeds for frozen BN #4761

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 2 commits into from
Oct 27, 2021
Merged

Conversation

datumbox
Copy link
Contributor

@datumbox datumbox commented Oct 27, 2021

Resolves partially #4506

Checked locally the first 10k seeds but can't make the test fail. Might be extremely rare that it fails.

Let's see if fixing the seed does the trick.

cc @pmeier

@facebook-github-bot
Copy link

facebook-github-bot commented Oct 27, 2021

💊 CI failures summary and remediations

As of commit f072a8e (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.

Copy link
Member

@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.

excellent seed choice

@datumbox datumbox merged commit 5ffba76 into pytorch:main Oct 27, 2021
@datumbox datumbox deleted the tests/flaky_frozenbn branch October 27, 2021 14:54
facebook-github-bot pushed a commit that referenced this pull request Nov 1, 2021
Reviewed By: datumbox

Differential Revision: D32064702

fbshipit-source-id: 9f7f6d3ffb346b85d34052326d6b5d1ad1ebba90
cyyever pushed a commit to cyyever/vision that referenced this pull request Nov 16, 2021
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