search

AcademySoftwareFoundation / OpenColorIO

A color management framework for visual effects and animation.
https://opencolorio.org
BSD 3-Clause "New" or "Revised" License
1.79k stars 454 forks source link

PyPI package contains dev dependencies #1811

Closed beatreichenbach closed 1 year ago

beatreichenbach commented 1 year ago

The current PyPI package contains development dependencies that add a lot of unneeded data for everyday installations.

Currently the setup includes tests and test data that is not needed by normal users and should only be made available for developers.

Could it be possible to consider an approach with optional dependencies, where setuptools is used to specify dev dependencies? For example: pip install opencolorio[dev]

If that is too much work, I think it would make more sense to just include the dev dependencies in the github repo and not distribute it to every installation to keep the package more light weight.

remia commented 1 year ago

Thanks for reporting @beatreichenbach! We are currently working on updating the Python wheels to include the command line apps, as part of that work the layout of the package is going to be updated and test code / data removed, this is happening in #1802.

I think we can expect wheels roughly half the size after this is completed.

beatreichenbach commented 1 year ago

Amazing! Thank you for your work on updating that and looking forward to the new lean packages. 👍