Skip to content

Pre-requisites for ubuntu install #2

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

jphuart
Copy link

@jphuart jphuart commented Oct 8, 2020

I've added into the install.sh script comments on the needed pre-requisites packages to ensure that the SAW works on a Ubuntu 20.04 OS. Maybe it could be improved adding how to do it for other distributions.

I've modified the swatplus_aw.sh script to use the correct modules and to match the functioning of the .bat script.

@jphuart
Copy link
Author

jphuart commented Oct 16, 2020

Last error was on the weather generator database, I've downloaded from bitbucket SWAT+ editor repository.

@jphuart jphuart closed this Oct 16, 2020
@jphuart jphuart reopened this Oct 16, 2020
@celray
Copy link
Owner

celray commented Apr 23, 2021

Do you think we should change to $HOME instead of $SWAT_HOME in the update for the latest SWAT+ release?

There are still some tests I have to do but I still have time to adapt it if $HOME Makes more sense

@jphuart
Copy link
Author

jphuart commented Apr 23, 2021

I was surprised about your message, now I understand what happens. I'm trying to define the easiest way of using this project in an isolated conda environment. $HOME variable is not suitable for this, I wanted then add a variable that can be set to the path of the conda environment path. I've called in a first test SWAT_HOME, which is a mistake, at least it should be called SWAT_AW_HOME. By the way, what I'm doing is not ready yet, I will continue on this in may. I will propose a proper pull request when it will work fine.

@celray
Copy link
Owner

celray commented Apr 23, 2021

Perfect, SWAT_AW_HOME sounds good. I will keep an eye on this. Thanks.

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.

2 participants