Skip to content

only set random generator if shuffle=true #5135

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
Jan 2, 2022
Merged

only set random generator if shuffle=true #5135

merged 2 commits into from
Jan 2, 2022

Conversation

xiaohu2015
Copy link
Contributor

@xiaohu2015 xiaohu2015 commented Dec 29, 2021

This pr is about #5051

set random_generator only when shuffle=true, just like https://pytorch.org/docs/stable/_modules/torch/utils/data/distributed.html#DistributedSampler

cc @datumbox

@facebook-github-bot
Copy link

facebook-github-bot commented Dec 29, 2021

💊 CI failures summary and remediations

As of commit 4aaa362 (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
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, thanks @xiaohu2015.

@datumbox datumbox merged commit e65a857 into pytorch:main Jan 2, 2022
facebook-github-bot pushed a commit that referenced this pull request Jan 5, 2022
Summary: Co-authored-by: Vasilis Vryniotis <[email protected]>

Reviewed By: datumbox, prabhat00155

Differential Revision: D33405361

fbshipit-source-id: 83c93424be8c68327e6bdaee5ef726f3d0c0ab33
facebook-github-bot pushed a commit that referenced this pull request Jan 6, 2022
…5135)

Differential Revision:
D33405361

Original commit changeset: 83c93424be8c

Original Phabricator Diff: D33405361

fbshipit-source-id: c7a9d50e4a83030b0744e0ea8b5a8066c94f4880
facebook-github-bot pushed a commit that referenced this pull request Jan 8, 2022
Summary: Co-authored-by: Vasilis Vryniotis <[email protected]>

Reviewed By: sallysyw

Differential Revision: D33479280

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