Although Lukas said to me in a recent discussion that all contributors use Macs, you must have all solved the tensorflow requirements issue that I and a few others have faced when trying to install THINGSvision on a Macbook. I have to get around this by cloning the repo and manually editing setup.py to allow for tensorflow-macos instead of plain tensorflow and running python setup.py install manually.
This PR adds conditions to both requirements.txt and setup.py to reflect conditional dependency specification (à laPEP508) based on the OS (so the installation doesn't fail if the user has a sufficient version of tensorflow-macos).
This would make it easier for me to keep up to date with the fast-moving pace of this library and hopefully for others who might struggle with the same issues I've faced.
Although Lukas said to me in a recent discussion that all contributors use Macs, you must have all solved the tensorflow requirements issue that I and a few others have faced when trying to install THINGSvision on a Macbook. I have to get around this by cloning the repo and manually editing
setup.pyto allow fortensorflow-macosinstead of plaintensorflowand runningpython setup.py installmanually.This PR adds conditions to both
requirements.txtandsetup.pyto reflect conditional dependency specification (à la PEP508) based on the OS (so the installation doesn't fail if the user has a sufficient version oftensorflow-macos).This would make it easier for me to keep up to date with the fast-moving pace of this library and hopefully for others who might struggle with the same issues I've faced.