-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add 2020.1.2 to wflow versions #268
Conversation
@sverhoeven The model loads and completes the model.setup() step. However the model does not complete initialization due to a possible GDAL_DATA environment variable error. From wflow log: This error does not occur with wflow v.2020.1.1 |
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Hmm, I tried to run docs/examples/wflow.ipynb with Docker container and it works, but with singularity fails. Seems GDAL_DATA is set to I hardcoded the GDAL_DATA and PROJ_LIB env in the Dockerfile. The new Docker image is available on Docker Hub and new sif file (ewatercycle-wflow-grpc4bmi_2020.1.2.sif) is on dcache. Now docs/examples/wflow.ipynb works with singularity as well. |
@sverhoeven This fix solved the issue. The scientific output looks good. However, the model.finalize() function does not work correctly. log: Creating the "outsum" folder manually in the output directory and rerunning model.finalize() solves the issue. This could be a local permission error. For now I think the PR can be merged. |
@jeromaerts thanks for testing. |
Kudos, SonarCloud Quality Gate passed! |
Can be tested by installing this branch locally and running
To prevent version warning when getting a wflow parameter set, add the version in ewatercycle.yaml.