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

When DDP num_workers = 0 was forced #19

Merged
merged 3 commits into from
Aug 23, 2021
Merged

When DDP num_workers = 0 was forced #19

merged 3 commits into from
Aug 23, 2021

Conversation

lvoegtlin
Copy link
Contributor

Description

When using the ddp the number of workers for the datamodule was set to 0. This slowed down the training by a factor of #amount_of_workers

How to Test/Run?

Old

python run.py experiment=development_baby_unet_cb55_10 trainer.max_epochs=3 test=false datamodule.num_workers=0

New

python run.py experiment=development_baby_unet_cb55_10 trainer.max_epochs=3 test=false

@lvoegtlin lvoegtlin added the Pipeline The general Hydra system label Jul 15, 2021
@lvoegtlin lvoegtlin added this to the 0.1.0 milestone Jul 15, 2021
@lvoegtlin lvoegtlin requested a review from powl7 July 15, 2021 09:13
@lvoegtlin lvoegtlin self-assigned this Jul 15, 2021
@lvoegtlin lvoegtlin added the bug Something isn't working label Jul 15, 2021
Copy link
Contributor

@powl7 powl7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sehr schnell! Awesome!

@powl7 powl7 merged commit 0adcbd4 into dev Aug 23, 2021
@powl7 powl7 deleted the dev_speed_up_training branch August 23, 2021 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Pipeline The general Hydra system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants