Skip to content

Fix length of DispatchDataLoader#175

Merged
sgugger merged 1 commit intomainfrom
fix_dispatch_dl_len
Sep 27, 2021
Merged

Fix length of DispatchDataLoader#175
sgugger merged 1 commit intomainfrom
fix_dispatch_dl_len

Conversation

@sgugger
Copy link
Copy Markdown
Collaborator

@sgugger sgugger commented Sep 27, 2021

Currently, the DispatchDataLoader object as the same length as the underlying dataloader, even if it will return 8 times less batches.

@sgugger sgugger requested a review from LysandreJik September 27, 2021 14:36
Copy link
Copy Markdown
Member

@LysandreJik LysandreJik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sgugger sgugger merged commit 929e17d into main Sep 27, 2021
@sgugger sgugger deleted the fix_dispatch_dl_len branch September 27, 2021 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants