Skip to content
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

Added single-loop examples to 08a and 08b (EnKF) #19

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

Conversation

burggraaff
Copy link
Contributor

I've added examples to both of the EnKF scripts, 08a and 08b, showing how to implement the EnKF assimilation in a single loop, which is how one would typically do it in practice. This may make it easier to translate the educational example to practical applications.

The branch to be pulled is already up-to-date with the current master branch. Only changes to the markdown, code, and new outputs are included; the notebook metadata are unchanged. All code has been tested.

@ZhiyiLiao
Copy link

In the example 11, "Wofost72_PP_notebook_11.conf" is needed, but this file is not included in this project. Could you please add this file, or direct a path to download? Thanks for your project, and it is useful.

@ajwdewit
Copy link
Owner

ajwdewit commented Jan 14, 2024 via email

@ZhiyiLiao
Copy link

ZhiyiLiao commented Jan 14, 2024 via email

1 similar comment
@ZhiyiLiao
Copy link

ZhiyiLiao commented Dec 31, 2024 via email

@mnqoliveira
Copy link

Hi, I don't know if this was by design, but some of the functions for use in the loop (script 8a) are using the definitions of the previous steps (e.g. H). Thank you for the script!

@burggraaff
Copy link
Contributor Author

@mnqoliveira - are you referring to the version on the master branch or the version in this pull request?

@mnqoliveira
Copy link

To the version in the pull request. I meant the final loop with the additional helper functions to run everything at once.

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.

4 participants