search

huggingface / datasets

🤗 The largest hub of ready-to-use datasets for ML models with fast, easy-to-use and efficient data manipulation tools
https://huggingface.co/docs/datasets
Apache License 2.0
19.29k stars 2.7k forks source link

handle config_name=None in push_to_hub #7229

Closed alex-hh closed 1 month ago

alex-hh commented 1 month ago

This caught me out - thought it might be better to explicitly handle None?

lhoestq commented 1 month ago

not sure it's a good idea, we always need a config name so better have the correct default and not support None (which could lead to think it doesn't have a config name, while it does)