Skip to content

Fix seed for TestColorJitter #4762

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
Merged

Conversation

NicolasHug
Copy link
Member

@NicolasHug NicolasHug commented Oct 27, 2021

@facebook-github-bot
Copy link

facebook-github-bot commented Oct 27, 2021

💊 CI failures summary and remediations

As of commit fe5e449 (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 15:36
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, as discussed offline change seed range to 10 prior merge.

@NicolasHug NicolasHug merged commit 50dfe20 into pytorch:main Oct 27, 2021
facebook-github-bot pushed a commit that referenced this pull request Nov 1, 2021
Summary:
* setting 100 seeds

* back to 10 seeds

Reviewed By: datumbox

Differential Revision: D32064707

fbshipit-source-id: 4cbe0883257f949b146c08f627c51e4160a14cda
cyyever pushed a commit to cyyever/vision that referenced this pull request Nov 16, 2021
* setting 100 seeds

* back to 10 seeds
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