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

Expose iterator-local generator to RandomSampler instance #65857

Closed
wants to merge 2 commits into from

Conversation

ejguan
Copy link
Contributor

@ejguan ejguan commented Sep 29, 2021

Stack from ghstack:

Make sure the iter-local generator is attached to Sampler. Then, the state of iter-local generator is able to be serialized and resumed.

Differential Revision: D31288686

@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Sep 29, 2021

🔗 Helpful links

💊 CI failures summary and remediations

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


  • 1/1 failures introduced in this PR

1 failure not recognized by patterns:

Job Step Action
GitHub Actions Lint / mypy Run mypy 🔁 rerun

This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

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

Click here to manually regenerate this comment.

@ejguan
Copy link
Contributor Author

ejguan commented Sep 29, 2021

@ejguan has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Make sure the iter-local generator is attached to Sampler. Then, the state of iter-local generator is able to be serialized and resumed.

Differential Revision: [D31288686](https://our.internmc.facebook.com/intern/diff/D31288686)

[ghstack-poisoned]
@ejguan ejguan added this to In Progress in torch.utils.data via automation Sep 29, 2021
ejguan added a commit that referenced this pull request Sep 29, 2021
ghstack-source-id: f1edb9c1c28ffb31c1f185f9f0d9da505b604f05
Pull Request resolved: #65857
@ejguan
Copy link
Contributor Author

ejguan commented Sep 29, 2021

@ejguan has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@github-actions
Copy link

Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as Stale.
Feel free to remove the Stale label if you feel this was a mistake.
Stale pull requests will automatically be closed 30 days after being marked Stale

@github-actions github-actions bot added the Stale label Nov 29, 2021
@github-actions github-actions bot closed this Apr 13, 2022
torch.utils.data automation moved this from In Progress to Done Apr 13, 2022
@facebook-github-bot facebook-github-bot deleted the gh/ejguan/89/head branch May 14, 2022 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants