Skip to content

Use nwm-ewts repository Python Package#121

Merged
cmaynard-ngwpc merged 3 commits intodevelopmentfrom
cmaynard_use_ewts_library
Mar 25, 2026
Merged

Use nwm-ewts repository Python Package#121
cmaynard-ngwpc merged 3 commits intodevelopmentfrom
cmaynard_use_ewts_library

Conversation

@mxkpp
Copy link

@mxkpp mxkpp commented Mar 18, 2026

The Error and Warning Trapping System has been refactored into a single repository that includes a Python package.
This update removes the previous package in this repository.

Additions

  • import the ewts package

Removals

  • The ngen Forcing repository nextgen_forcing_ewts package

Changes

  • Update git workflow for unit tests to install from the nwm-ewts repository python package

Testing

  1. Local testing with ngen and PW testing on DEV2 cluster

Checklist

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

Testing checklist

Target Environment support

  • Linux

@cmaynard-ngwpc cmaynard-ngwpc force-pushed the cmaynard_use_ewts_library branch from 640b031 to d1a4e55 Compare March 24, 2026 04:14
@cmaynard-ngwpc cmaynard-ngwpc changed the title EWTS Use nwm-ewts repository Python Package Mar 24, 2026
@cmaynard-ngwpc cmaynard-ngwpc marked this pull request as ready for review March 25, 2026 04:16
@cmaynard-ngwpc cmaynard-ngwpc merged commit 3836faf into development Mar 25, 2026
6 checks passed
@cmaynard-ngwpc cmaynard-ngwpc deleted the cmaynard_use_ewts_library branch March 25, 2026 04:17
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