Skip to content

Conversation

samanklesaria
Copy link
Collaborator

As the original use of PIL was to test ffmpeg related functionality, but that functionality was removed, we no longer need the PIL dependency or the utilities that use it. This PR removes these parts.

@meta-cla meta-cla bot added the CLA Signed label Sep 4, 2025
Copy link

pytorch-bot bot commented Sep 4, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/audio/4088

Note: Links to docs will display an error until the docs builds have been completed.

❌ 7 New Failures, 8 Pending

As of commit 8d55202 with merge base 2e30055 (image):

NEW FAILURES - The following jobs have failed:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

Copy link
Collaborator

@pearu pearu left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks, @samanklesaria!

@pearu pearu added this to the 2.9 milestone Sep 4, 2025
@samanklesaria samanklesaria marked this pull request as ready for review September 4, 2025 19:40
@samanklesaria samanklesaria requested a review from a team as a code owner September 4, 2025 19:40
@NicolasHug NicolasHug merged commit b60d685 into main Sep 5, 2025
43 checks passed
@NicolasHug NicolasHug deleted the remove_PIL branch September 5, 2025 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants