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

Enable GTS DA #35

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Conversation

yuanxue2870
Copy link

@yuanxue2870 yuanxue2870 commented Nov 15, 2024

Describe your changes

Summarise all code changes included in PR:
All edits are made in the DA_update submodule, which include:
1.Fix bugs for $WINLEN
2.Correct typos in the do_landDA.sh
3.Add Jiarui's current GTS yaml and add the elevation check (to be consistent with GHCN.yaml filters)
4.Enable 6-hr cycle for GTS DA

List any associated PRs in the submodules.
NOAA-PSL/land-DA_update#25

Issue ticket number and link

List the git Issue that this PR addresses:
#36

Test output

Is this PR expected to pass the DA_IMS_test (ie., does it change the output)?
Yes
Does it pass the DA_IMS_test?
Yes
If changes to the test results are expected, what are these changes? Provide a link to the output directory when running the test:
N/A

Checklist before requesting a review

  • My branch being merged is up to date with the latest develop.
  • I have performed a self-review of my code by examining the differences that will be merged.
  • I have not made any unnecessary code changes / changed any default behavior.
  • My code passes the DA_IMS_test, or differences can be explained.

@yuanxue2870
Copy link
Author

Please review: @ClaraDraper-NOAA @jiaruidong2017, Thank you!

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.

1 participant