Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skip jpeg comparison tests with PIL #5169

Merged
merged 2 commits into from
Jan 6, 2022

Conversation

NicolasHug
Copy link
Member

Linux and MacOS CIs are failing because the latest PIL 9 version now relies on libjpeg-turbo (it used to be the case only for Windows). Torchvision relies on libjpeg, not libjpeg-turbo, hence the comparison test failures.

We skip them in this PR to put CI back to green. This is only temporary until we find a better solution, tracked in #5162.

@facebook-github-bot
Copy link

facebook-github-bot commented Jan 6, 2022

💊 CI failures summary and remediations

As of commit 0b1e742 (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

Copy link
Collaborator

@pmeier pmeier left a comment

Choose a reason for hiding this comment

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

Thanks!

@NicolasHug NicolasHug merged commit 5dc61cb into pytorch:main Jan 6, 2022
facebook-github-bot pushed a commit that referenced this pull request Jan 8, 2022
Reviewed By: sallysyw

Differential Revision: D33479265

fbshipit-source-id: 2ac5eef5b9224f3c94687cf9921e0cf5bf7cbc9d
datumbox pushed a commit to datumbox/vision that referenced this pull request Jan 20, 2022
datumbox added a commit to datumbox/vision that referenced this pull request Jan 20, 2022
# Conflicts:
#	test/test_image.py
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