openfl icon indicating copy to clipboard operation
openfl copied to clipboard

Fix tutorials on Windows

Open itrushkin opened this issue 3 years ago • 2 comments

  • PyTorch DataLoader's num_workers usage is removed due to Windows errors/slowness with num_workers > 0. Closes #549.

itrushkin avatar Jan 24 '23 12:01 itrushkin

The merge of this PR may also fix the MemoryError on Windows.

itrushkin avatar Feb 06 '23 16:02 itrushkin

LGTM, but could we add this to the documentation that num_workers>0 can be used on ubuntu so that users have that option?

@mansishr I like the idea of documenting the ability to speed up the data loading but the num_workers issue is not related to OpenFL. It appears even in non-federated usage of DataLoader. Users always have the option of setting the num_workers because torch.utils.data.DataLoader class accepts it. Why do we need to tell users how to use PyTorch?

itrushkin avatar Feb 16 '23 16:02 itrushkin