NOAA-OWP / noah-owp-modular

Modularized version of the NOAH-MP land surface model.
Other
8 stars 19 forks source link

Add support for calibratable parameters (lumped version) #102

Closed GreyREvenson closed 5 months ago

GreyREvenson commented 6 months ago

Purpose

This PR exposes the model's calibratable parameters via its BMI so that they can be calibrated via NextGen. This PR exposes these parameters in the lumped (i.e., non-gridded) version of Noah-OWP-Modular.

Additions

Calibratable parameters

The following parameters were added to the model's BMI:

** FRZX and KDT were added because they are recalculated/reinitialized when setting SMCMAX, DKSAT and/or REFKDT. FRZX and KDT were added so that they may be included in the unit test program; they are not intended to be calibrated.

*** This PR adds SCAMAX to Noah-OWP-Modular because it is a NWM 3.0 calibratable parameter. SCAMAX was added as a member of parameters_type. The calculation of FSNO (i.e., snow covered area) was amended to follow WRF-Hydro's calculation as:

water%FSNO = parameters%SCAMAX * TANH( water%SNOWH /(2.5 * parameters%Z0 * FMELT))

Changes

The Noah-OWP-Modular unit test program (/test/noahowp_driver_test.f90) was updated to facilitate testing of the calibratable parameters.

Testing

The refactored unit test program was executed and gave these results.

Notes

This PR replicates PR #95, which exposed the calibratable parameters for the gridded version of Noah-OWP-Modular.

GreyREvenson commented 6 months ago

Ngen automatic integration tests may be failing due to NGEN issue #751.

drakest123 commented 6 months ago

PR #102 Review

As noted by Grey, the NGen integration tests indicate that a shared library called “register_bmi” is missing. I don't know the significance of these checks.

GreyREvenson commented 5 months ago

Thanks for your review, @drakest123. I put the BMI case listings in alphabetical order for easier reading.