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

Run ufs forecast in the ewok forecast suite #59

Closed
wants to merge 31 commits into from

Conversation

cmgas
Copy link
Collaborator

@cmgas cmgas commented Feb 7, 2023

Description

Draft PR to get started on the ewok forecast task discussion.
I know the files are a little repetitive with Dom's PR feature/ufs_dom and we should clean that up better later.
Based on @climbfuji 's work for easier comparison of the files, but I am not trying to merge in his branches.

You need first to build the ufs-bundle with Dom's branches:
https://github.com/JCSDA/ufs-bundle

Then, you need to checkout specific branches for the following repositories:

  • feature/forecast_ufs_develop for fv3-jedi (should only differ from develop in the ewok repository).
  • feature/forecast_ufs for fv3-jedi-data (should only differ from feature/ufs_dom in the UFSFILES repository)
  • feature/forecast_ufs for r2d2-data (adds c48 input files in gfs/an)
  • feature/forecast_ufs for ewok
  • develop for oops

You do not need to rebuild, since only the ewok repo is different in these. I opened the PR against develop for easy comparison.

Associated PRs:

The experiments are:
experiments/ufs-3dvar-c48-fc.yaml
experiments/ufs-3dvar-c48.yaml
experiments/ufs-forecast-c48.yaml

@cmgas cmgas self-assigned this Feb 7, 2023
Copy link
Contributor

@shlyaeva shlyaeva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are these changes needed for fv3-jedi tests, or for ewok runs?

@cmgas
Copy link
Collaborator Author

cmgas commented Feb 15, 2023

@shlyaeva for ewok runs, but these files will live somewhere else eventually (with static B files for example)

Copy link

@fcvdb fcvdb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EWOK is looking for files under testinput_tier_1/inputs/UFSfiles, but the directory in fv3-jedi-data is named UFSFILES.
Once I renamed the directory the forecast defined in experiments/ufs-forecast-c48.yaml successfully completed.

@cmgas cmgas changed the base branch from feature/ufs_dom to develop February 16, 2023 16:16
@fcvdb
Copy link

fcvdb commented Mar 2, 2023

The full EWOK with the ufs-bundle (Master PR #562 completed. The data are good to go.

@cmgas
Copy link
Collaborator Author

cmgas commented Mar 27, 2023

Not needed

@cmgas cmgas closed this Mar 27, 2023
@cmgas cmgas deleted the feature/forecast_ufs branch June 27, 2024 15:41
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.

6 participants