search

yandex / rep

Machine Learning toolbox for Humans
http://yandex.github.io/rep/
Other
689 stars 144 forks source link

script to install REP environment using conda #41

Closed anaderi closed 9 years ago

arogozhnikov commented 9 years ago
  1. Is it needed to have duplicate of .yaml with conda requirements?
  2. install-conda.sh and install-conda-env.sh are terribly close. If those cannot be merged, at least description would help to understand their difference.
  3. Why providing one more Dockerfile? (Docker is meant to have only one tested way of deployment, no?)

Occam is unhappy.

anaderi commented 9 years ago

Is it needed to have duplicate of .yaml with conda requirements?

those are not duplicate, those for py2 and py3.

install-conda.sh and install-conda-env.sh are terribly close. If those cannot be merged, at least description would help to understand their difference.

it is merged in install-repbase-env.sh

Why providing one more Dockerfile? (Docker is meant to have only one tested way of deployment, no?)

to speed-up things a bit. maybe later we'll end up with a single one.