facebookresearch / facebookresearch/SlowFast

DistributedSampler should have shuffle=shuffle

Open
#248 0 comments 1 reaction 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
7.4k
Forks
1.3k
PR merge metrics
No merged PRs in 30d

Description

Line 108 in slofast/datasets/loader.py should be `sampler = DistributedSampler(dataset, shuffle=shuffle) if cfg.NUM_GPUS > 1 else None`. In some instances, it is useful to run the data in a distributed, non-shuffled fashion. The DistributedSampler has `shuffle=True` by default.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.