-
Notifications
You must be signed in to change notification settings - Fork 1
Description
First of all, thank you for this software! A few recommendations to improve the docs:
I personally couldn't get the PyPy version to work and had to install from GitHub. It also wasn't immediately clear to me that this only works on Python 3.10 since the wheel is hard pinned to that version. Anyone on a newer version of Python will experience failures unless they take specific steps:
I recommend the docs be updated to inform people that they should
- Install micromamba (or similarly use miniconda) and then create an environment: e.g.
micromamba create -n colorenv python=3.10 -c nvidiathen activate that environmentmicromamba activate colorenv(this ensures the correct Python is installed) - Use GitHub install method and
pip install -r requirements/requirements.txtfrom within your environment from above
--
Also, in the GitHub install instructions, this file doesn't exist out of the box
pip install ../ColorTransferLib/dist/ColorTransferLib-2.0.3-py3-none-any.whl <Not sure why the ../ was in there and the name of the package wasn't matching what's in this directory
I think it's supposed to be
pip install dist/colortransferlib-2.0.1.post3-py3-none-any.whl