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

Stagger __init__ calls to reduce peak memory consumption #1543

Merged
merged 2 commits into from
Nov 9, 2020

Conversation

RobertLucian
Copy link
Member

@RobertLucian RobertLucian commented Nov 8, 2020

Closes #1450.


checklist:

  • run make test and make lint
  • test manually an example that has a failing/succeeding __init__ constructor

@RobertLucian RobertLucian requested a review from deliahu November 8, 2020 11:02
@RobertLucian RobertLucian changed the title Stagger __init__ calls to prevent getting OOM Stagger __init__ calls to reduce peak memory consumption Nov 8, 2020
@RobertLucian RobertLucian merged commit d64c756 into master Nov 9, 2020
@RobertLucian RobertLucian deleted the misc/stagger-init-processes branch November 9, 2020 11:36
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.

Stagger __init__() when processes_per_replica > 1 by default
2 participants