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

fix problem of 'accelerator.is_main_process' to run in mutiple GPUs #5340

Merged
merged 1 commit into from
Oct 10, 2023
Merged

fix problem of 'accelerator.is_main_process' to run in mutiple GPUs #5340

merged 1 commit into from
Oct 10, 2023

Conversation

jiaqiw09
Copy link
Contributor

@jiaqiw09 jiaqiw09 commented Oct 9, 2023

What does this PR do?

fix #5338

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?
  • Did you read our philosophy doc (important for complex PRs)?
  • 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. not required
  • Did you write any new necessary tests? not required

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.

@patrickvonplaten

@jiaqiw09
Copy link
Contributor Author

jiaqiw09 commented Oct 9, 2023

@DN6 here is the PR.

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@jiaqiw09
Copy link
Contributor Author

@DN6 @patrickvonplaten all tests passed, is this PR still under review? And any suggestion for this PR?

Best wishes!

@DN6 DN6 merged commit 9c82b68 into huggingface:main Oct 10, 2023
11 checks passed
AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this pull request Apr 26, 2024
…uggingface#5340)

fix problem of 'accelerator.is_main_process' to run in mutiple GPUs or NPUs

Co-authored-by: jiaqiw <wangjiaqi50@huawei.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug on checkpoints_total_limit when runing mutiple GPUs
3 participants