Skip to content

Fix typos related to conda stuff#206

Merged
ocefpaf merged 6 commits intooceanhackweek:sourcefrom
NoraLoose:fix-conda-typos
Aug 17, 2022
Merged

Fix typos related to conda stuff#206
ocefpaf merged 6 commits intooceanhackweek:sourcefrom
NoraLoose:fix-conda-typos

Conversation

@NoraLoose
Copy link
Copy Markdown
Member

This fixes three minor typos related to conda instructions on the website, including the link to the conda environment file.

@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 17, 2022

Deploy Preview for oceanhackweek-preview ready!

Name Link
🔨 Latest commit 387aec0
🔍 Latest deploy log https://app.netlify.com/sites/oceanhackweek-preview/deploys/62fd2b9e991f8400081a8472
😎 Deploy Preview https://deploy-preview-206--oceanhackweek-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Comment thread _InstructionSiteUpdates.md Outdated
Co-authored-by: Filipe <ocefpaf@gmail.com>
Comment thread resources/prep/conda.md Outdated
## Conda on your own computer

Conda may be used on your computer as well as the Hub. If you wish to install the same environment as the hub is running, after you install Conda, you can download the [`environment.yml`](https://github.com/oceanhackweek/ohw20-tutorials/blob/master/environment.yml) that we use, then `conda create -n <ENV NAME> --file environment.yml`
Conda may be used on your computer as well as the Hub. If you wish to install the same environment as the hub is running, after you install Conda, you can download the [`environment.yml`](https://github.com/oceanhackweek/jupyter-image/blob/main/py-base/environment.yml) that we use, then `conda env create -n <ENV NAME> --file environment.yml`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This env file is Linux only. Let's keep the reference to the ohw20-tutorials and I'll add a proper env file there. We still need to change master to main.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good. The link was broken, but there may be a better link to use here, as you say. Thanks, Filipe!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We may not have a great option here, for now. The OHW20 env file includes both Python and R, which we may not want keep here. But to at least fix the broken link, here's the correct, updated link:
https://github.com/oceanhackweek/ohw-tutorials/blob/OHW20/environment.yml

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @emiliom, I updated the link!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just added a "canonical one" here: oceanhackweek/ohw-tutorials#100

It should be in main so we can port from year to year. Also, I'm pretty sure that one from OHW20 is also platform specific. The one I added above should be platform agnostic, even on the new mac M1/2.

@emiliom
Copy link
Copy Markdown
Member

emiliom commented Aug 17, 2022

Thanks for submitting these fixes @NoraLoose !

Comment thread resources/prep/conda.md Outdated
Co-authored-by: Filipe <ocefpaf@gmail.com>
@ocefpaf ocefpaf merged commit cc07167 into oceanhackweek:source Aug 17, 2022
@NoraLoose NoraLoose deleted the fix-conda-typos branch August 17, 2022 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants