Skip to content

Conversation

@plaguss
Copy link
Contributor

@plaguss plaguss commented Jan 24, 2025

Description

Fixes #1109, by checking for columns with images only when PIL is available.

@plaguss plaguss self-assigned this Jan 24, 2025
@plaguss plaguss requested a review from gabrielmbmb January 24, 2025 13:01
@github-actions
Copy link

Documentation for this PR has been built. You can view it at: https://distilabel.argilla.io/pr-1112/

@plaguss plaguss linked an issue Jan 24, 2025 that may be closed by this pull request
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 24, 2025

CodSpeed Performance Report

Merging #1112 will not alter performance

Comparing fix-pil-in-push-to-hub (12421bb) with main (e746527)

Summary

✅ 1 untouched benchmarks

@plaguss plaguss changed the base branch from develop to main January 27, 2025 11:44
@gabrielmbmb gabrielmbmb merged commit 067b3d7 into main Jan 28, 2025
4 of 8 checks passed
@gabrielmbmb gabrielmbmb deleted the fix-pil-in-push-to-hub branch January 28, 2025 09:24
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.

[BUG] push_to_hub method requires PIL to be installed

2 participants