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.

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

Suggest installing CLIMADA via conda in installation instructions #714

Merged
merged 9 commits into from
May 15, 2023

Conversation

peanutfun
Copy link
Member

@peanutfun peanutfun commented May 4, 2023

Changes proposed in this PR:

  • Instead of building the climada_env and installing CLIMADA via pip, the "Simple Instructions" in the installation instructions now advise users to simply install CLIMADA via conda. This only requires a single command.
  • Advertise installation with conda also in the README.md.

This PR fixes #673

PR Author Checklist

PR Reviewer Checklist

@peanutfun peanutfun changed the base branch from main to develop May 4, 2023 15:44
README.md Outdated Show resolved Hide resolved
doc/guide/install.rst Outdated Show resolved Hide resolved
@chahank
Copy link
Member

chahank commented May 8, 2023

Thanks for the conda install @peanutfun !

@bguillod
Copy link
Collaborator

bguillod commented May 8, 2023

Out of curiosity: Why do we now want to recommend to users to install CLIMADA via conda, which is (in my experience at least) much more cumbersome than via pip?

@chahank
Copy link
Member

chahank commented May 8, 2023

CLIMADA can only really be installed using conda at the moment (the climada_env needs to be made with conda). So I am not sure to understand what you mean?

doc/guide/install.rst Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@peanutfun
Copy link
Member Author

@chahank Did you finish your review?

@chahank
Copy link
Member

chahank commented May 12, 2023

Yes, thanks!

@peanutfun peanutfun merged commit 0548d95 into develop May 15, 2023
@emanuel-schmid emanuel-schmid deleted the install-conda-package branch May 25, 2023 08:40
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.

Make conda install climada the default installation instruction
3 participants