Skip to content

Add Daft to list of integrated libraries for datasets#1892

Merged
davanstrien merged 3 commits into
huggingface:mainfrom
kevinzwang:daft-datasets-docs
Sep 9, 2025
Merged

Add Daft to list of integrated libraries for datasets#1892
davanstrien merged 3 commits into
huggingface:mainfrom
kevinzwang:daft-datasets-docs

Conversation

@kevinzwang

@kevinzwang kevinzwang commented Sep 3, 2025

Copy link
Copy Markdown
Contributor

Daft now has native integration for reading and writing Hugging Face datasets, which we have documented in our docs pages! This PR adds the corresponding docs to the Hugging Face docs.

This PR relies on the daft_hf.png file being uploaded to the huggingface/documentation-images repo, which I have made a PR for here. The image:
daft_hf

@davanstrien davanstrien left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool! I think it could be worth making the distinction between pushing to the Hub vs writing to a dataset.Dataset a little bit more explicit but this is looking very nice otherwise 🤗

Comment thread docs/hub/datasets-daft.md Outdated
Comment thread docs/hub/datasets-daft.md Outdated
Comment thread docs/hub/datasets-libraries.md Outdated
@lhoestq

lhoestq commented Sep 4, 2025

Copy link
Copy Markdown
Member

awesome ! it would also be cool to showcase how to load/push multimodal data

@kevinzwang

Copy link
Copy Markdown
Contributor Author

awesome ! it would also be cool to showcase how to load/push multimodal data

@lhoestq thanks for the feedback! I'm not yet happy about Daft's story around the Hugging Face Dataset multimodal types (images, video, audio) and they are currently just represented as some nested binary type in Daft. I plan on building better integrations for them soon, and once we have that I'd love to update these docs with more info if that works.

@lhoestq

lhoestq commented Sep 5, 2025

Copy link
Copy Markdown
Member

sounds good, looking forward for the multimodal integration :) feel free to ping me if you have questions on the structure or if I can help

@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

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.

@kevinzwang

kevinzwang commented Sep 5, 2025

Copy link
Copy Markdown
Contributor Author

@davanstrien this (and the documentation images PR) is ready to merge on my end. I don't have the permissions for it, so feel free to do so!

@pcuenca pcuenca left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please rebase or fix the conflicts so the PR only shows the pertinent changes @kevinzwang? 🙏

@kevinzwang

Copy link
Copy Markdown
Contributor Author

Could you please rebase or fix the conflicts so the PR only shows the pertinent changes @kevinzwang? 🙏

@pcuenca done!

@pcuenca pcuenca left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good for me whenever @davanstrien is happy with it.

Comment thread docs/hub/datasets-daft.md Outdated
Comment thread docs/hub/datasets-daft.md Outdated
Co-authored-by: Pedro Cuenca <pedro@huggingface.co>
Comment thread docs/hub/datasets-daft.md Outdated
@davanstrien davanstrien merged commit a7bc6b4 into huggingface:main Sep 9, 2025
1 check passed
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.

5 participants