Merge memory-refactor-pytestUpdate into refactor dev branch#99
Closed
aufdenkampe wants to merge 42 commits into
Closed
Merge memory-refactor-pytestUpdate into refactor dev branch#99aufdenkampe wants to merge 42 commits into
memory-refactor-pytestUpdate into refactor dev branch#99aufdenkampe wants to merge 42 commits into
Conversation
…hydrologyTeam/ClearWater-modules into memory-refactor-pytestUpdate
…e/TSM coarse mesh example
…ne_mesh example; also created a new linked example notebook
It seems that the relative pathing now all works on modules, but I haven't tested on Windows. @jrutyna, please test. Related to EcohydrologyTeam/ClearWater-riverine#133
…e text in the '03' notebook
aufdenkampe
added this to the
v0.4: Refactor Modules Framework for Scalability milestone
Specifies the architectural plan for clearwater_modules_v3, the merged codebase that combines v2's framework (Process class composition, YAML configuration, per-process substepping, chunking execution) with v1's optimization and correctness work (kernel optimization, hotstart, wet-mask gating, latent-heat unit fix, thin-water stability guard). Covers: thin-overlay package strategy, component inventory for TSM, YAML backward compatibility, migration plan, v2 retirement at v3 0.1.0 ship and v1+v2 removal at v3 1.0.0, phased implementation plan with deliverables, test infrastructure including v2/v3 parity tests, performance targets calibrated to the verified Sumwere Creek baseline (89s for 4,320 timesteps), risk register, and open design questions for review. Target: v3 1.0.0 with TSM and legacy NSM1 capabilities by 2026-05-31. NSM2 features (multi-pool OM, alkalinity/pH, methane/sulfide, silica) are out of scope for v3 1.0.0 and will be added incrementally in subsequent v3 releases.
Member
Author
|
Closing without merging, in favor of: This excludes commit b21e973, which was added prematurely. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Includes: