Fix the issue where `set_epoch` does not take effect. by hongjx175 · Pull Request #3556 · huggingface/accelerate
What does this PR do?
Fixes # (issue)
When training for multiple epochs, the set_epoch defined in the Sampler does not work because set_epoch in DataLoaderShard is missing a further level call.
Before submitting
- This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
- Did you read the contributor guideline,
Pull Request section? - Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case. - Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings. - Did you write any new necessary tests?
Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@SunMarc @zach-huggingface
