AlignmentResearch / KataGoVisualizer

MIT License
3 stars 1 forks source link

Remove python-dev #89

Closed ed1d1a8d closed 1 year ago

ed1d1a8d commented 1 year ago

python-dev no longer exists in the latest versions of debian, see e.g. the build failure here https://app.circleci.com/pipelines/github/AlignmentResearch/KataGoVisualizer/653/workflows/c09a6929-e51a-4458-8b5f-eeee32dba983/jobs/1316

I've removed python-dev (which we never needed cause the container comes with python 3.10), and also adjusted the pip install process to work better. Namely, I merged the tensorflow install line with the requirements line. Installing tensorflow separately would also upgrade protobuf to a version that is incompatible with streamlit. The --use-pep517 is to address https://github.com/pypa/pip/issues/8559.

netlify[bot] commented 1 year ago

Deploy Preview for goattack canceled.

Name Link
Latest commit 0c67719446e9313b8094b3c12c46577391021acf
Latest deploy log https://app.netlify.com/sites/goattack/deploys/6491eb9017621e0007522bee