-
Notifications
You must be signed in to change notification settings - Fork 6
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
base: master
Are you sure you want to change the base?
Conversation
Last error was on the weather generator database, I've downloaded from bitbucket SWAT+ editor repository. |
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 |
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. |
Perfect, SWAT_AW_HOME sounds good. I will keep an eye on this. Thanks. |
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.