deepspeedai / deepspeedai/DeepSpeed

Ensure DeepSpeedDataLoader matches torch DataLoader semantics

Open
#176 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
Python
Stars
43.1k
Forks
5k
Avg merge
4d 15h
Merged PRs (30d)
112

Description

DeepSpeedDataLoader is a wrapper for torch DataLoader and should at a minimum match the semantics. Right now this property is not well tested and discrepancies have already been identified, such as this #167. In my estimation, a fair bit of work is required to achieve this goal, including the following items:

  1. Support batching and non-batching modes.

  2. Unit tests to validate expected behavior

  3. Performance comparison between DeepSpeedDataLoader and torch DataLoader.

  4. Some documentation of DeepSpeedDataLoader, including any semantic deviation from torch DataLoader.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Locate the DeepSpeedDataLoader implementation and compare its behavior with the torch DataLoader documentation, including the discrepancy referenced by #167. Define tests for batching and non-batching semantics, add the requested performance comparison and documentation, and consider the work complete when supported behavior and any deviations are documented and validated.

Written by the indexing model from the issue text.

Assessment

Tech stack
python, pytorch
Domain
data, machine-learning
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.