Make setup.py compatible with pipenv by apoorvaeternity · Pull Request #4121 · huggingface/diffusers (original) (raw)
What does this PR do?
This PR makes setup.py
compatible with pipenv
. Currently, installation of diffusers
from source using pipenv
is broken (see: pypa/pipenv#5167 (comment)) and fails with the error:AttributeError: 'Subscript' object has no attribute 's'
The same fix was recently merged in huggingface/transformers#24789
Before submitting
- This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
- Did you read the contributor guideline?
- Did you read our philosophy doc (important for complex PRs)?
- Was this discussed/approved via a Github issue or the forum? Please add a link to it if that's the case.
- Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings. - Did you write any new necessary tests?
Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.