Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

blog post about anaconda #428

Merged
merged 22 commits into from
Aug 12, 2024
Merged

blog post about anaconda #428

merged 22 commits into from
Aug 12, 2024

Conversation

nabobalis
Copy link
Contributor

Fixes #427

dpshelio
dpshelio reviewed Aug 10, 2024
View reviewed changes
Copy link
Member

@dpshelio dpshelio left a comment

Choose a reason for hiding this comment

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

Thanks @nabobalis!! That was quick and good. I've added a couple of suggestions.

posts/2024/2024-08-09.md Outdated Show resolved Hide resolved
posts/2024/2024-08-09.md Outdated Show resolved Hide resolved
posts/2024/2024-08-09.md Outdated Show resolved Hide resolved
posts/2024/2024-08-09.md Outdated Show resolved Hide resolved
posts/2024/2024-08-09.md Outdated Show resolved Hide resolved
posts/2024/2024-08-09.md Outdated Show resolved Hide resolved
@nabobalis @dpshelio
Apply suggestions from code review
2a21ec4 
Co-authored-by: David Pérez-Suárez <d.perez-suarez@ucl.ac.uk>
nabobalis
nabobalis commented Aug 10, 2024
View reviewed changes
posts/2024/2024-08-09.md Outdated Show resolved Hide resolved
nabobalis
nabobalis commented Aug 10, 2024
View reviewed changes
posts/2024/2024-08-09.md Outdated Show resolved Hide resolved
nabobalis
nabobalis commented Aug 10, 2024
View reviewed changes
posts/2024/2024-08-09.md Outdated Show resolved Hide resolved
nabobalis
nabobalis commented Aug 10, 2024
View reviewed changes
posts/2024/2024-08-09.md Outdated Show resolved Hide resolved
@nabobalis nabobalis closed this Aug 10, 2024
@nabobalis nabobalis reopened this Aug 10, 2024
dpshelio
dpshelio approved these changes Aug 10, 2024
View reviewed changes
Copy link
Member

@dpshelio dpshelio left a comment

Choose a reason for hiding this comment

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

I introduced not good wording which I'm suggesting to fix it now.

posts/2024/2024-08-09-anaconda.md Outdated Show resolved Hide resolved
@nabobalis @dpshelio
Update posts/2024/2024-08-09-anaconda.md
ee0aace 
Co-authored-by: David Pérez-Suárez <d.perez-suarez@ucl.ac.uk>
Cadair
Cadair reviewed Aug 11, 2024
View reviewed changes
posts/2024/2024-08-09-anaconda.md Outdated Show resolved Hide resolved
posts/2024/2024-08-09-anaconda.md Outdated Show resolved Hide resolved
posts/2024/2024-08-09-anaconda.md Outdated
Anything that is "curated, built, maintained, and served by Anaconda's engineers on its secure cloud infrastructure", has these licenses.

So to be very clear.
Using the defaults channel can lead to you becoming legally required to pay Anaconda for the use of `conda` and the defaults channel.
Copy link
Member

Choose a reason for hiding this comment

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

Put this in italic or emphasis instead.

posts/2024/2024-08-09-anaconda.md Outdated

The answer from the SunPy Project is the following:

- Use [miniforge](https://github.com/conda-forge/miniforge) which is a `conda` distribution that uses the "conda-forge" channel, a channel that does not have these restrictions.
Copy link
Member

Choose a reason for hiding this comment

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

I think it's worth (somewhere) expanding on what conda-forge is, specifically that it's a community project and it's a bad idea to mix the two?

posts/2024/2024-08-09-anaconda.md Outdated
Though, you could get rid of the default channel(s) using the `conda config` command as shown [in this Stack Overflow answer about switching channels from anaconda to conda-forge](https://stackoverflow.com/a/67708768).
This is good for new environments but it doesn't remove what you've got already installed (e.g., the `base` environment) and you may be still infringing the Terms of Services for Anaconda.

Therefore, the cleanest way would be to remove Anaconda completely and install [miniforge](https://github.com/conda-forge/miniforge).
Copy link
Member

Choose a reason for hiding this comment

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

Rather than linking to the shitty miniforge GH page why not link to our install page?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

posts/2024/2024-08-09-anaconda.md Outdated Show resolved Hide resolved
posts/2024/2024-08-09-anaconda.md Show resolved Hide resolved
posts/2024/2024-08-09-anaconda.md Outdated Show resolved Hide resolved
posts/2024/2024-08-09-anaconda.md Outdated

## What is not free to use?

- Anaconda's default channel, which is installed by default when you either install the `Anaconda` distribution or the `miniconda` distribution.
Copy link
Member

Choose a reason for hiding this comment

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

I would reword this to give slightly more context for what the channel is. Rather than saying "installed by default" say something like "where packages are installed from by default".

posts/2024/2024-08-09-anaconda.md Outdated Show resolved Hide resolved
@nabobalis @Cadair
Apply suggestions from code review
e2b0ef9 
Co-authored-by: Stuart Mumford <stuart@cadair.com>
@dpshelio dpshelio merged commit 9c81cf9 into main Aug 12, 2024
5 checks passed
@dpshelio dpshelio deleted the blog branch August 12, 2024 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Cadair Cadair Cadair left review comments

@dpshelio dpshelio dpshelio approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add a post about how to freely use conda
3 participants
@nabobalis @dpshelio @Cadair