search

eugenevinitsky / sequential_social_dilemma_games

Repo for reproduction of sequential social dilemmas
MIT License
380 stars 134 forks source link

Requirements fixes #163

Closed internetcoffeephone closed 4 years ago

internetcoffeephone commented 4 years ago

Added wheel to requirements.txt, without it pip install -r requirements.txt throws errors if it is not present already.

Froze lz4 library version requirement.

internetcoffeephone commented 4 years ago

After some further research, I found some more issues: there is an issue with venv/pip where linux distros don't update their base python installation pip. tensorflow==2.0.0 cannot install with older versions of pip and will throw an error.

Since you can't upgrade pip and install the new tensorflow from the same requirements.txt (I tried, it throws the same error as it does when using an old pip and not upgrading pip), perhaps it's worth mentioning/adding to the readme that the following command needs to be executed after creation of a venv: pip3 install --upgrade pip setuptools wheel

Then, the wheel requirement is no longer necessary.