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

Running ngen on CONUS with Routing: Build, Realization Configs, Routing Config, and Documentation #805

Merged
merged 4 commits into from
May 10, 2024

Conversation

stcui007
Copy link
Contributor

@stcui007 stcui007 commented Apr 26, 2024

This PR adds several realization config files and the routing config file for running ngen with routing on CONUS. It also provides specific documentation on how to build and run MPI jobs with routing in ngen framework,

Additions

Realization config files for runing routing after noah-owp-modular, running routing after noah+pet+cfe, running routing
after noah+pet+smp+sft+cfe, all on CONUS.

Also the configuration file for running routing on CONUS: routing_config_CONUS.yaml.

Updated documentation.

Removals

Changes

Testing

Running ngen tests with all 3 realization configs.

Screenshots

Notes

This PR should be merged after PR#775 and PR#794. Although no real conflict, it is better for understanding the construction of the realization configs and the documentation.

Todos

Checklist

  • [x ] PR has an informative and human-readable title
  • [x ] Changes are limited to a single goal (no scope creep)
  • [x ] Code can be automatically merged (no conflicts)
  • [x ] Code follows project standards (link if applicable)
  • [x ] Passes all existing automated tests
  • [x ] Any change in functionality is tested
  • [x ] New functions are documented (with a description, list of inputs, and expected output)
  • [x ] Placeholder code is flagged / future todos are captured in comments
  • [x ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
  • [x ] Reviewers requested with the Reviewers tool ➡️

Testing checklist (automated report can be put here)

Target Environment support

  • [x ] Linux

@stcui007 stcui007 force-pushed the conus_w_routing branch from a5ce2ba to d46f033 Compare May 1, 2024 14:20
@stcui007 stcui007 force-pushed the conus_w_routing branch from d46f033 to 09b8007 Compare May 1, 2024 14:44
@stcui007 stcui007 marked this pull request as ready for review May 2, 2024 13:25
doc/NextGen_ON_CONUS.md Outdated Show resolved Hide resolved
doc/NextGen_ON_CONUS.md Outdated Show resolved Hide resolved
doc/NextGen_ON_CONUS.md Outdated Show resolved Hide resolved
doc/NextGen_ON_CONUS.md Outdated Show resolved Hide resolved
doc/NextGen_ON_CONUS.md Outdated Show resolved Hide resolved
doc/NextGen_ON_CONUS.md Outdated Show resolved Hide resolved
doc/NextGen_ON_CONUS.md Outdated Show resolved Hide resolved
@stcui007
Copy link
Contributor Author

stcui007 commented May 6, 2024 via email

@program-- program-- merged commit 28f5885 into NOAA-OWP:master May 10, 2024
19 checks passed
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.

2 participants