-
Notifications
You must be signed in to change notification settings - Fork 45
Update installation instructions to use miniforge/mamba #491
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Conversation
for more information, see https://pre-commit.ci
👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below. |
A couple of links to NOAA NCEI are failing. These are due to impacts from Hurricane Helene and should eventually work again. |
Ignoring the expected link check failures in order to finally get this in. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great @ktyle, just one small recommended addition.
|
||
1. It's quicker and takes up less disk space. | ||
2. It encourages you to install only the packages you need in reproducible isolated environments for specific projects. This is generally a more robust way to work with open source tools. | ||
3. It uses `conda-forge` as the default channel for packages. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great, but might not mean much to the newbie. Here's my suggested addition:
3. It uses `conda-forge` as the default channel for packages. | |
3. It uses `conda-forge` as the default channel, which is our recommended way to get up-to-date, interoperable packages. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, looks like I clicked the merge button too quickly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no prob, i'll commit and merge the suggestion by @brian-rose.
Addresses #391