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

Remove Julia from environment.yml file #193

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

samgdotson
Copy link
Collaborator

I couldn't find a guide for contributors, so I hope this is okay. This PR contains no new features but will provide users with a more consistent experience across platforms.

Summary

This PR changes the default environment setup by removing the Julia installation from the environment.yml file. I also updated the section for installing PRAS on Linxu/Mac to install Julia via conda. This change was motivated by Issue #169 and my own experience installing ReEDS with the current documentation.

This change provides the following benefits:

  1. A consistent user experience. Since PRAS is optional, Julia should be optional for users on all platforms.
  2. ReEDS installation can now be automated on Windows machines, where before it was impossible (or prohibitively difficult).
  3. Retains the ability for a single line installation on Unix machines.
conda env create -f environment.yml; conda activate reeds2; conda install -y "julia>=1.8"
  1. Reduces additional documentation. My original fix for this was to add instructions to setup.md in the Windows section. This approach is much more streamlined.
     

Technical Details

Issues resolved (if any)

Closes #169

Known incompatibilities (if any)

None

Relevant sources or documentation (if any)

Per this stackoverflow post, Julia is not available on Windows via Conda. It is not clear when this will be changed, if ever.

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.

Installation
1 participant