Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Enhancement: Add PIL to runtime dependencies #5586

Closed
2 tasks done
BigRoy opened this issue Sep 6, 2023 · 1 comment
Closed
2 tasks done

Enhancement: Add PIL to runtime dependencies #5586

BigRoy opened this issue Sep 6, 2023 · 1 comment
Labels
community contribution type: enhancement Enhancements to existing functionality

Comments

@BigRoy
Copy link
Collaborator

BigRoy commented Sep 6, 2023

Is there an existing issue for this?

  • I have searched the existing issues.

Please describe the feature you have in mind and explain what the current shortcomings are?

In testing the new version (beta) of a supported host by OpenPype I noticed an upgrade to Python 3.10 within the host. Upon launching the host the OpenPype integration does not load correctly (and also breaks other bootstrapping of the host) due to OpenPype providing its PIL library on the PYTHONPATH which is incompatible with that Python version. The host has some scripts of its own which try to import the Pillow library, fail and thus break the startup procedure (even for some host-native tools).

How would you imagine the implementation of the feature?

Similar to other Python-version dependent dependencies this might need to become a runtime dependency, like ocio and otio with this PR #5206

Are there any labels you wish to add?

  • I have added the relevant labels to the enhancement request.

Describe alternatives you've considered:

If the environment would be fully configured through rez (like ynput/ayon-core#816) more complex approaches could be taken where PIL does get provided for hosts who don't provide it themselves and we have a matching PIL library available for the Python version of that host version.

Additional context:

Due to the bootstrapping procedure the host also receives no registered host (install_host does not trigger).

[cuID:OP-6756]

@BigRoy
Copy link
Collaborator Author

BigRoy commented Jul 29, 2024

This is somewhat solved in AYON with runtime dependencies, etc. - not perfect and there are still issues with AYON and Houdini but there are already issues open for that. I'll close this and we can continue on AYON somewhere when needed with a new issue.

@BigRoy BigRoy closed this as completed Jul 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
community contribution type: enhancement Enhancements to existing functionality
Projects
None yet
Development

No branches or pull requests

2 participants