search

thedataincubator / forecaster

3 stars 1 forks source link

Install instructions #1

Closed ZachGlassman closed 5 years ago

ZachGlassman commented 6 years ago

Please alter the README to include instructions regarding

  1. installing the local python environment
  2. running local tests
sdorsher commented 6 years ago

I am not currently working on this, I did this installation by hand. I ignored "detect-secrets" and "mock". I did not upgrade my conda version itself.

However, it is not possible to do this simply using either cat conda_requirements.txt | conda install or conda install < conda_requirements.txt

Some line by line script is needed. Does anyone remember how we did this in the flask framework? I can't find the instructions.

Otherwise, it's possible to write a script. It's okay to modify the way that I structured the bokeh version, without closing the issue, if necessary. It is not easy to write a bash script that parses line by line. I suggest python.

ZachGlassman commented 6 years ago

Please open a pull request with the proposed changes to the readme. There is a flag you can pass to conda to allow installation from a file.

sdorsher commented 6 years ago

conda install --file conda_requirements.txt conda install --file requirements.txt conda install --file dev-requirements.txt

sdorsher commented 6 years ago

Done, though I think I screwed it up the first time. Thank you.

Steven

On Jun 21, 2018, at 3:13 PM, ZachGlassman notifications@github.com wrote:

Please open a pull request with the proposed changes to the readme. There is a flag you can pass to conda to allow installation from a file.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/thedataincubator/forecaster/issues/1#issuecomment-399228044, or mute the thread https://github.com/notifications/unsubscribe-auth/AKXkESZrQUWM6idse9jUnMbBqXcPaaGCks5t-_5bgaJpZM4UqGab.

ZachGlassman commented 6 years ago

please open a pull request with these changes in the readme. Additionally, files in requirements.txt should be installed with pip

sdorsher commented 6 years ago

How do I do that? I don’t see an option to specify how I open a pull request.

Steven

On Jun 21, 2018, at 3:25 PM, ZachGlassman notifications@github.com wrote:

please open a pull request with these changes in the readme. Additionally, files in requirements.txt should be installed with pip

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/thedataincubator/forecaster/issues/1#issuecomment-399231506, or mute the thread https://github.com/notifications/unsubscribe-auth/AKXkEQaHUVTWGf33QjVq_9Eqn0WemfSMks5t_AFJgaJpZM4UqGab.

ZachGlassman commented 6 years ago

@sdorsher Please work on the bokeh upgrade, @josylman please take this. The install instructions should be in the README

sdorsher commented 6 years ago

cat conda-requirements.txt | conda install

I can't figure out why this doesn't work. Honestly.

ZachGlassman commented 6 years ago

there is an option to install from a file

sdorsher commented 5 years ago

I can’t alter the readme if you won’t approve my changes. Issue closed.