update readme with new install and test info #60
Conversation
|
|
||
| conda activate cloudside | ||
| conda install metar --channel=conda-forge | ||
| conda create --name=cloudside python=3.8 notebook pytest pandas matplotlib requests coverage |
There was a problem hiding this comment.
I think we should add metar in this and keep the conda-forge specification
There was a problem hiding this comment.
since we are testing against pip install metar and there are no fancy dependency binaries, why would we advise users to install via conda and/or conda-forge?
There was a problem hiding this comment.
it seems that where projects can be pure pip, they should be and the tests prove that users don't need conda-forge to make this work.
There was a problem hiding this comment.
I kind of see what you mean, but the partial list of dependencies (missing metar) feels off to me. In my corner of the python world, people conda are now mostly installing via miniforge for mambaforge, where the defaults channel is omitted from .condarc as that's going to a subscription model.
Easiest thing would be to drop conda instructions entirely.
No description provided.