Skip to content

Refactor of Supplemental Precipitation Mod (suppPrecipMod.py) - #134

Merged
mxkpp merged 13 commits into
NGWPC-7625_PI_10_ngen_forcing_refactorfrom
suppPrecipMod
Aug 13, 2026
Merged

Refactor of Supplemental Precipitation Mod (suppPrecipMod.py)#134
mxkpp merged 13 commits into
NGWPC-7625_PI_10_ngen_forcing_refactorfrom
suppPrecipMod

Conversation

@mdeshotel

Copy link
Copy Markdown

This PR is a refactor of suppPrecipMod.py that mimics the refactor that was done for forcingInputMod.py.

Additions

Removals

Changes

-Refactor suppPrecipMod.py to mimic the refactor that was done for forcingInputMod.py.

Testing

Screenshots

Notes

Todos

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
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future todos are captured in comments
  • Visually tested in supported browsers and devices (see checklist below 👇)
  • Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
  • Reviewers requested with the Reviewers tool ➡️

Testing checklist

Target Environment support

  • Linux

@mdeshotel
mdeshotel marked this pull request as draft April 7, 2026 18:33
@mxkpp mxkpp changed the title Supp precip mod Refactor of Supplemental Precipitation Mod (suppPrecipMod.py) Aug 10, 2026
@mxkpp

mxkpp commented Aug 11, 2026

Copy link
Copy Markdown

Reviewed logic paths to confirm original behavior is equivalent to new behavior. Added some fixes as shown in commits. Rebased against development.

mxkpp added 2 commits August 10, 2026 22:48
…y, travel recursively instead of falling back to pure equality tests after initial shallow inspection. This allows nested objects to have the numerical tolerances applied as intended.
@mxkpp
mxkpp requested a review from idtodd August 11, 2026 03:19
@mxkpp

mxkpp commented Aug 11, 2026

Copy link
Copy Markdown

All ngen-forcing pytests are passing (with tolerance) from the nwm-rte Dev Container:

( cd src/ngen-forcing && pytest )
( cd src/ngen-forcing && mpirun -n 2 pytest )

@idtodd I needed to make significant changes. This is ready for your review.

@mxkpp
mxkpp marked this pull request as ready for review August 11, 2026 03:19
@mxkpp
mxkpp changed the base branch from development to NGWPC-7625_PI_10_ngen_forcing_refactor August 13, 2026 18:14
@mxkpp

mxkpp commented Aug 13, 2026

Copy link
Copy Markdown

Merging to the new refactor branch which was branched off from development today.

@mxkpp
mxkpp merged commit 2aaee61 into NGWPC-7625_PI_10_ngen_forcing_refactor Aug 13, 2026
7 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.

3 participants