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

SIMClr does not respect number of workers #318

Closed
Haydnspass opened this issue Oct 29, 2020 · 1 comment · Fixed by #329
Closed

SIMClr does not respect number of workers #318

Haydnspass opened this issue Oct 29, 2020 · 1 comment · Fixed by #329
Assignees
Labels
help wanted Extra attention is needed
Milestone

Comments

@Haydnspass
Copy link

🐛 Bug

The SIMClr example does not respect the number of workers or at least not as expected.

I suspect that this is due to the changed behaviour of pytorch 1.7 where the default num_threads is #physical processors whereas pytorch 1.6 had a default of 1.

The actual number of cores being used in this notebook is num_threads * num_workers, while this might be technically true it is highly unexpected (isn't it?).

To Reproduce

Steps to reproduce the behavior:

  1. Install pytorch_lightning >1 with torch=1.7
  2. Runt he SIMClr example with varying amount of workers
  3. Monitor the CPU utilisation.

Expected behavior

Number of cores beuing utilised = number of workers

Environment

  • PyTorch Version 1.7
  • OS (e.g., Linux): Linux
  • How you installed PyTorch (conda, pip, source): conda + pip
  • Python version: 3.8
  • CUDA/cuDNN version: 11
  • GPU models and configuration: RTX 2080 TI
@Haydnspass Haydnspass added the help wanted Extra attention is needed label Oct 29, 2020
@github-actions
Copy link

Hi! thanks for your contribution!, great first issue!

@ananyahjha93 ananyahjha93 mentioned this issue Nov 2, 2020
8 tasks
@ananyahjha93 ananyahjha93 self-assigned this Nov 3, 2020
@Borda Borda added this to the v0.3 milestone Jan 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants