earthenterprise
earthenterprise copied to clipboard
run_tutorial.sh should use default asset root
The run_tutorial.sh script uses a non-standard asset root, requiring the user to change the asset root before running the script, which is tedious, unnecessary, and confusing. Since the script is often used for more than simply testing, it has caused some confusing issues because users do not expect the asset root path to be changed from its default /gevol/assets
. The scripts are in earth_enterprise/src/fusion/tools/gee_test
.
Acceptance Criteria:
- The run_tutorial.sh, run_historic.sh, status.sh, status_historical.sh, publish_tutorial.sh, and publish_historical.sh scripts should use the default asset root
- Update the README file to remove instructions telling users to change asset root
Stretch goals:
- Add an optional parameter to change the asset root path
- Add an optional parameter to change the publish root path
@tst-ccamp we need to revisit this ticket. The scripts named above don't change the asset root - the user must do that before running them. That may turn this into a simple documentation change. We need to decide what we want the outcome of this ticket to be in light of this.
The asset root should be the default of /gevol/assets
rather than /usr/local/google/gevol_test/assets
We should update the README as well because it tells users to change the asset root.
I'm not sure that stretch goal 3 makes sense because the script doesn't change the asset root, so there's nothing to change back.
@tst-lsavoie I've accepted your suggestions