Skip to content
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

Replace set_deterministic with non-deprecated spelling #4212

Merged
merged 1 commit into from
Jul 27, 2021
Merged

Conversation

ezyang
Copy link
Contributor

@ezyang ezyang commented Jul 27, 2021

No description provided.

Signed-off-by: Edward Z. Yang <ezyang@fb.com>
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.

LGTM if green, thanks!

@ezyang
Copy link
Contributor Author

ezyang commented Jul 27, 2021

failing tests look spurious

@ezyang ezyang merged commit 30fd10b into master Jul 27, 2021
@github-actions
Copy link

Hey @ezyang!

You merged this PR, but no labels were added.

@fmassa fmassa deleted the deterministic branch July 28, 2021 13:05
facebook-github-bot pushed a commit that referenced this pull request Aug 3, 2021
Summary: Signed-off-by: Edward Z. Yang <ezyang@fb.com>

Reviewed By: NicolasHug

Differential Revision: D30069952

fbshipit-source-id: 7f4f2bf6e8e6fd47a29f947d780c540577c62ae5
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