Sdg23 update installation quick start page #440
Conversation
pages/installation_quick_start.md
Outdated
|
|
||
| ## Conda ## | ||
| ## Mamba ## |
There was a problem hiding this comment.
Should probably still contain the word "conda" because that's what people know.
| ## Mamba ## | |
| ## mamba/conda ## |
| If you don't have [conda][] installed yet, follow the [installation | ||
| instructions for conda][]. | ||
|
|
There was a problem hiding this comment.
I think we should still have links to installing mamba/miniforge (see MDAnalysis/UserGuide#361 (comment) ) and conda. There are people who don't use it every day. It's important to give some minimal guidance. It's also a bit odd to recommend mamba but requiring people to somehow have conda already installed.
| MDAnalysis via [mamba][] supports only serial calculations. | ||
| For OpenMP-based parallelism, use [pip][] and ensure you have |
There was a problem hiding this comment.
Is this still true? @hmacdope ?
|
I'd say, look at the User Guide: Contributing / with conda-forge packages instructions for mamba installation. We recently rewrote those for the UGM and they look reasonable. Do NOT follow the standard UG conda installation instructions because they are not correct MDAnalysis/UserGuide#405 |
1ae92cc to
48bad02
Compare
|
Thanks @orbeckst for your comments. I incorporated them. I kept it simple as this is the quick start guide, but happy to take more suggestions as to what level of detail should be included here. |
There was a problem hiding this comment.
LGTM, thanks!
@jennaswa @micaela-matta do you also want to have a look or should we merge?
To be reviewed by @micaela-matta and @orbeckst.
This PR fixes #385.