KEMBAR78
Add albumentations to use dataset by ternaus · Pull Request #7596 · huggingface/datasets · GitHub
Skip to content

Conversation

@ternaus
Copy link
Contributor

@ternaus ternaus commented Jun 5, 2025

  1. Fixed broken link to the list of transforms in torchvison.
  2. Extended section about video image augmentations with an example from Albumentations.

@ternaus
Copy link
Contributor Author

ternaus commented Jun 6, 2025

@lhoestq ping

1 similar comment
@ternaus
Copy link
Contributor Author

ternaus commented Jun 9, 2025

@lhoestq ping

Copy link
Member

@lhoestq lhoestq left a comment

Choose a reason for hiding this comment

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

LGTM ! sorry for the delay ^^'

I think we can remove the torchvision example altogether tbh and only keep albumentations, could you remove it before we merge ?

ternaus added 4 commits June 15, 2025 15:07
…entation

- Replaced torchvision transforms with Albumentations for image augmentation.
- Renumbered sections for clarity and updated descriptions accordingly.
- Emphasized key points for using Albumentations with 🤗 Datasets.
@ternaus ternaus force-pushed the add_albumentations_to_use_dataset branch from 716d805 to 9e4da2e Compare June 15, 2025 22:09
- Added a new key point emphasizing that the dataset is ready for training with machine learning frameworks.
- Ensured clarity by maintaining consistent formatting and structure.
@ternaus
Copy link
Contributor Author

ternaus commented Jun 15, 2025

@lhoestq Thanks. Cleaned up torchvision.

Copy link
Member

@lhoestq lhoestq left a comment

Choose a reason for hiding this comment

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

thanks :)

@lhoestq lhoestq merged commit 660c645 into huggingface:main Jun 17, 2025
@ternaus ternaus deleted the add_albumentations_to_use_dataset branch June 17, 2025 18:38
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.

2 participants