Document HDF5 support by klamike · Pull Request #7740 · huggingface/datasets
Navigation Menu
{{ message }}
huggingface / datasets Public
- Notifications You must be signed in to change notification settings
- Fork 3.1k
Merged
lhoestq merged 3 commits intohuggingface:mainfrom
Sep 24, 2025Merged
lhoestq merged 3 commits intohuggingface:mainfrom
lhoestq merged 3 commits intohuggingface:mainfrom
Conversation
Copy link
Contributor
I think these are at least the main places where we should put content. Ideally it is not just repeated in the final version
ref #7690
- Wait for Refactor HDF5 and preserve tree structure #7743 to land
klamike commented Aug 19, 2025
docs/source/tabular_load.mdx
Outdated
Show resolved
Hide resolved
docs/source/tabular_load.mdx Outdated Show resolved Hide resolved
This was referenced
Aug 19, 2025Merged
Copy link
Contributor Author
klamike
commented
Sep 3, 2025
klamike commented
Sep 3, 2025@lhoestq any guidance on what else to add/feedback on what is there now? It seems a bit minimal, but I don't think it's worth doing an entire page on HDF5?
lhoestq
marked this pull request as ready for review
lhoestq approved these changes Sep 24, 2025
Copy link
Member
lhoestq
left a comment
lhoestq
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good as a first iteration !
I also added the related classes to the docs
lhoestq
merged commit
5dc1a17
into
huggingface:main
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants