-
Notifications
You must be signed in to change notification settings - Fork 12
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
Update installation documentation #69
Conversation
Should we add a yml environment file that matches these install instructions? Both for users and for use in the dockerfile for testing. I have one that is similar but not identical that I have been using while writing the dockerfile, but it would probably be best to have it match the install guide. We could also add a little paragraph mentioning that the conda and pip stuff can be obtained using the yml file. |
Oh, that's a good idea. I'll add a YML file soon. |
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.
Thanks for making these changes - just a few lingering suggestions and/or questions for discussion
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.
Thanks @connoramoreno
Rewrites the installation instructions to reflect updates to dependencies. Also streamlines the installation process to reduce the number of steps. Confirmed to work with a new Conda test environment.
Closes #62.