Skip to content

Commit

Permalink
perf(data): explicitly set val_sampler=None
Browse files Browse the repository at this point in the history
  • Loading branch information
zjykzj committed May 27, 2022
1 parent e1d8c73 commit c46bc96
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions simpleir/data/build.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,9 @@

from yacs.config import CfgNode
from torch.utils.data import IterableDataset, DataLoader, Sampler
from torch.utils.data.distributed import DistributedSampler

from zcls2.data.transform.build import build_transform
from zcls2.data.sampler.build import build_sampler

from .dataloader.build import build_dataloader
from .dataset.build import build_dataset
Expand All @@ -26,11 +26,13 @@ def build_data(cfg: CfgNode, w_path: bool = False) -> Tuple[Sampler, DataLoader,
val_transform, val_target_transform = build_transform(cfg, is_train=False)
val_dataset = build_dataset(cfg, val_transform, val_target_transform, is_train=False)

train_sampler = None
val_sampler = None
if isinstance(train_dataset, IterableDataset):
train_sampler, val_sampler = None, None
shuffle = False
else:
train_sampler, val_sampler = build_sampler(cfg, train_dataset, val_dataset)
if cfg.DISTRIBUTED:
train_sampler = DistributedSampler(train_dataset)
shuffle = train_sampler is None

train_loader, val_loader = build_dataloader(cfg,
Expand Down

0 comments on commit c46bc96

Please sign in to comment.