-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Fix NonMatchingSplitsSizesError/ExpectedMoreSplits when passing data_dir/data_files in no-code Hub datasets #6925
Conversation
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
Do you think this is worth making a patch release for? |
I will add some regression tests before merging. And I will make a patch release afterwards. |
Show benchmarksPyArrow==8.0.0 Show updated benchmarks!Benchmark: benchmark_array_xd.json
Benchmark: benchmark_getitem_100B.json
Benchmark: benchmark_indices_mapping.json
Benchmark: benchmark_iterating.json
Benchmark: benchmark_map_filter.json
Show updated benchmarks!Benchmark: benchmark_array_xd.json
Benchmark: benchmark_getitem_100B.json
Benchmark: benchmark_indices_mapping.json
Benchmark: benchmark_iterating.json
Benchmark: benchmark_map_filter.json
|
…asets when passing data_dir/data_files (#6925) * Do not use exported dataset infos in some cases * Add regression tests
…asets when passing data_dir/data_files (#6925) * Do not use exported dataset infos in some cases * Add regression tests
I'm hitting this error now, using Spaces. Here's what an attempt to just get the 'validation' split is doing: code:
log:
|
Hi Meg ! Thanks for reporting, I'll see how I can fix this. In the meantime feel free to pass |
Fix
NonMatchingSplitsSizesError
orExpectedMoreSplits
error for no-code Hub datasets if the user passes:data_dir
data_files
The proposed solution is to avoid using exported dataset info (from Parquet exports) in these cases.
Additionally, also if the user passes
revision
other than "main" (so that no network requests are made).This PR fixes a bug introduced by:
Fix #6918, fix #6939.