Skip to content
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.

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

surfacing a problem in current LoadHF #1599

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dafnapension
Copy link
Collaborator

@dafnapension dafnapension commented Feb 11, 2025

It realizes too late that streaming should be False. For both cases: split = str, and split=None

@dafnapension dafnapension force-pushed the touch_the_loaded_dataset branch from 06d6513 to 5189d7e Compare February 12, 2025 19:12
@dafnapension dafnapension changed the title touch the dataset to make sure loaded correctly with the streaming value you chose surfacing a problem in current LoadHF Feb 12, 2025
@dafnapension dafnapension force-pushed the touch_the_loaded_dataset branch 5 times, most recently from 1c29423 to dc4c920 Compare February 18, 2025 19:19
…ing should be False. For both cases: split = str, and split=None

Signed-off-by: dafnapension <[email protected]>
@dafnapension dafnapension force-pushed the touch_the_loaded_dataset branch from dc4c920 to ac251bd Compare February 19, 2025 07:24
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.

1 participant