generated from NOAA-OWP/owp-open-source-project-template
-
Notifications
You must be signed in to change notification settings - Fork 30
NGWPC NWM PI-3 Formulation Selection Delivery 2025-11-21 #162
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
Open
cmaynard-ngwpc
wants to merge
95
commits into
NOAA-OWP:master
Choose a base branch
from
NGWPC:ngwpc-3.1.2.0.0
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
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
update submodule url to gitlab See merge request NGWPC/nwm-ngen/cfe!1
fix to CFE unit test for mass balance check See merge request NGWPC/nwm-ngen/cfe!2
…e() function in bmi_cfe.c
# Conflicts: # src/bmi_cfe.c
Adding logging mechanism to CFE module See merge request NGWPC/nwm-ngen/cfe!3
This reverts merge request !3
Revert "Merge branch 'NGWPC-3889' into 'development'" See merge request NGWPC/nwm-ngen/cfe!4
recommit of logger changes for CFE module See merge request NGWPC/nwm-ngen/cfe!5
NGWPC-4428 - CFE logging changes See merge request NGWPC/nwm-ngen/cfe!6
rc-master: Release 1.0.0 Merge See merge request NGWPC/nwm-ngen/cfe!7
Logger optimization See merge request NGWPC/nwm-ngen/cfe!8
Added release number, date and commit hash of official release See merge request NGWPC/nwm-ngen/cfe!9
Adding feature for dynamically setting log level systemwide See merge request NGWPC/nwm-ngen/cfe!10
Logger optimization See merge request NGWPC/nwm-ngen/cfe!11
This reverts commit a7e8451.
This reverts commit 2945134.
Revert "changed the way logLevel char pointer is checked for NULL" See merge request NGWPC/nwm-ngen/cfe!12
Development Branch Merge to release-candidate branch for Release rc-1.1.0 See merge request NGWPC/nwm-ngen/cfe!13
Refactor and add env var checking See merge request NGWPC/nwm-ngen/cfe!35
Remove placholder comments See merge request NGWPC/nwm-ngen/cfe!36
Merge development into release-candidate for release 3.1.1.3.0-rc1 See merge request NGWPC/nwm-ngen/cfe!37
…ster # Conflicts: # .github/workflows/add_to_project.yml # INSTALL.md # LICENSE # configs/README.md # configs/cfe1.0/cfe_config_cat_87_pass.txt # configs/cfe_config_cat_87.txt # configs/cfe_config_laramie_pass_aet_rz.txt # include/bmi_cfe.h # include/cfe.h # include/nash_cascade.h # realizations/realization_cfe_pet_surfnash.json # src/bmi_cfe.c # src/cfe.c # src/nash_cascade.c # test/main_unit_test.c
Merge OWP Github master branch See merge request NGWPC/nwm-ngen/cfe!39
Created serialization methods for CFE state data See merge request NGWPC/nwm-ngen/cfe!38
Change malloc to calloc to ensure null pointers
Fix log level from SEVERE to WARNING for a warning
fix flux larger than storage error
Merge release-candidate into development for release 3.1.1.3.0-rc2
Merge development into release-candidate for release 3.1.1.4.0-rc1
Changed warning msg per OWP request
…ure_profile as "1" so UDUNITS2 can parse them
Merge updated warning per OWP to the development branch
Express dimensionless units of ice_fraction_xinanjiang and soil_moisture_profile as "1" so UDUNITS2 can parse them (NGWPC-7604)
Adapt serialization to revised protocol
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Extensive enhancements and cleanup across the CFE model and its configuration, testing, and logging systems. Major functional updates include improvements to runoff, infiltration, Nash cascade parameters, AET calculations, and soil moisture handling, along with numerous fixes for unit consistency, memory allocation issues, and error conditions. The logging framework was standardized and refined, including level corrections and reduced noise from warnings. Initial serialization support for model state was added and aligned with a revised protocol. Significant cleanup efforts removed unused or outdated files, updated documentation, improved config structures, and ensured compatibility with UDUNITS2. Continuous integration workflows were streamlined, and tests, realization files, and configuration defaults were updated for correctness and consistency. Overall, the changes increase model stability, correctness, and maintainability while aligning the codebase with current standards and workflows.