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

Conda package should support pillow-simd #4103

Open
bryant1410 opened this issue Jun 23, 2021 · 1 comment
Open

Conda package should support pillow-simd #4103

bryant1410 opened this issue Jun 23, 2021 · 1 comment

Comments

@bryant1410
Copy link
Contributor

bryant1410 commented Jun 23, 2021

🐛 Bug

torchvision Conda package, at least with v0.10.0, depends on pillow, which complicates using pillow-SIMD.

To Reproduce

Steps to reproduce the behavior:

  1. Install torchvision with Conda.
  2. Try to uninstall pillow, but would try to uninstall torchvision. Alternatively, you can do conda remove --force pillow to only remove it, but the environment will be in an inconsistent state. It's the best workaround AFAIK though.
  3. Install pillow-SIMD.

Expected behavior

The expected behavior would be to be able to choose between pillow and pillow-SIMD, or be able to uninstall pillow without breaking things, and then install pillow-SIMD. After this, every time you try installing something with conda, it's gonna try to install pillow again.

Maybe pillow shouldn't be a dependency then? Maybe the ideal solution is to pillow-SIMD to provide a conda package called pillow, or something like that (or like a virtual package to support both?).

@bryant1410
Copy link
Contributor Author

Another workaround: uploadcare/pillow-simd#44 (comment) (creating a conda package of pillow-SIMD from the source called pillow). Though it didn't work fine for me. Seems to require more configuration changes.

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

No branches or pull requests

2 participants