Skip to content
This repository has been archived by the owner on Aug 18, 2020. It is now read-only.

Fixed PILLOW_VERSION import error with pillow v7.0 #81

Closed
wants to merge 2 commits into from
Closed

Fixed PILLOW_VERSION import error with pillow v7.0 #81

wants to merge 2 commits into from

Conversation

akashpalrecha
Copy link
Contributor

torchvision import breaks with pillow's recent update to v7.0.0.

Release notes for pillow v7.0.0 say: PILLOW_VERSION has been removed. Use __version__ instead.

Issue: torchvision tries to import PILLOW_VERSION which causes problems.

Fixed this by constraining pillow version to 6.2.1 in the environment.yml file. (conda does not yet have 6.2.2)

This is a workaround until a new version of torchvision is officially released.

More info here:
pytorch/vision#1712
pytorch/vision#1712 (comment)

`torchvision` import breaks with pillow's recent update to v7.0.0.
Fixed by constraining pillow version to 6.2.1 (`conda` does not yet have 6.2.2)
More info here: 
pytorch/vision#1712
pytorch/vision#1712 (comment)
This will allow `conda` to solve the environment in a less constrained way as opposed to fixing the version for `pillow` in the `environment.yml` file.
@sgugger sgugger closed this Feb 7, 2020
@sgugger
Copy link
Contributor

sgugger commented Feb 7, 2020

No, we do not want to constrain pillow. There is a proper version of torchvision that deals with the import problem, if we did a constrain, it would be on this.

@akashpalrecha
Copy link
Contributor Author

akashpalrecha commented Feb 7, 2020

I see. Just saw torchvision 0.5.0 deals with this already. (https://github.com/pytorch/vision/releases)

My bad. Should’ve checked properly.

Is this change going to be made soon or are there checks to be made before an updated version of torchvision can be used ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants