Skip to content

Fix len() for lddl dataloader #1168

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
Wesley-Jzy wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix len() for lddl dataloader #1168

Wesley-Jzy wants to merge 1 commit into from

Conversation

@Wesley-Jzy
Copy link

@Wesley-Jzy Wesley-Jzy commented Jul 2, 2022

When using dataloader to load my own dataset, I found there was no return in len() for non-ParquetDataset. I guess it's a bug.
@Wesley-Jzy Wesley-Jzy changed the title fix: add rern for lddl dataloader fix: add return for lddl dataloader Jul 2, 2022
@Wesley-Jzy Wesley-Jzy changed the title fix: add return for lddl dataloader Fix len() for lddl dataloader Jul 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@Wesley-Jzy