Skip to content

Borland Updates March 21, 2022#265

Merged
jhs507 merged 48 commits intomasterfrom
borland_updates_mar_21_22
Apr 12, 2022
Merged

Borland Updates March 21, 2022#265
jhs507 merged 48 commits intomasterfrom
borland_updates_mar_21_22

Conversation

@jhs507
Copy link
Collaborator

@jhs507 jhs507 commented Apr 6, 2022

Updates were made to the Borland source to correct various issues. These changes need to be made in the crhmcode version and merged in.

@jhs507 jhs507 marked this pull request as draft April 6, 2022 20:30
@jhs507
Copy link
Collaborator Author

jhs507 commented Apr 6, 2022

List of changes also found in notes.txt

CRHM 08/11/21

Add fallstat_correction parameter in Volumetric module to allow correction for the estimated fallstat, with default value as 1.

if(fallstat_correction[hh] >= 0.0)
   fallstat[hh] = fallstat[hh]*fallstat_correction[hh];

CRHM 03/18/22

In Canopy, CanopyClearing, and CanopyClearingGap module, removing Gamma Distribution Correction term (1.0 + 3.0/Alpha + 2.0/sqr(Alpha))
term in mean particle mass equation, correction as,
Mpm = 4.0/3.0 * M_PI * PBSM_constants::DICE * RadiusRadiusRadius;

In evap module, /86400 to ra term for rs/ra in Penman-Monteith method, because rs is in d/m while ra is in s/m, correction as,

   evap[hh] = (delta(hru_t[hh])*Q*Global::Freq + (RHOa*Cp/
              (lambda(hru_t[hh])*1e3)*(estar(hru_t[hh]) - hru_ea[hh])/(ra/86400)))/
              (delta(hru_t[hh]) + gamma(Pa[hh], hru_t[hh])*(1.0 + rs[hh]/(ra/86400)))/
              Global::Freq;

CRHM 03/21/22

In Canopy, CanopyClearing, and CanopyClearingGap module, invert the time step conversion term in weekly dimensionless unloading coefficient,
use unloading rate coefficient U = -log(c)/t for snow unloading determined by inverse function of c = e^(-Ut) = 0.678 based on Eq. 14 in Hedstrom and Pomeroy (1998)
correction as,
const float U = -1log(0.678)/(247*Global::Freq/24);

   else if(IceBulbT < unload_t[hh]) // has to be at least one interval. Trip on half step
        SUnload[hh] = Snow_load[hh]*U; // the dimensionless unloading coefficient already /interval, 21Mar2022 correction: use unloading rate coefficient U

CRHM 04/06/22

Update description for parameters, variables and units in the modules below:

longVt
netall
evap_Resist
evapD_Resist
ShuttleWallace
ShuttleWallaceD
crack
PrairieInfiltration
Ayers
Greencrack
GreenAmpt
frozen
frozenAyers
Soil
evapX
SoilX
SoilDetention
SoilPrairie
glacier
SWESlope
Netroute
Netroute_D
Netroute_M
Netroute_M_D
REW_route
REW_route2
Canopy
CanopyClearing
CanopyClearingGap
NeedleLeaf
XG
Volumetric
tsurface
K_Estimate
interception
lake_evap  

jhs507 added 25 commits April 7, 2022 11:28
…ch when set multiplies fallstat after its calclulation. It defaults to 1.
Removing Gamma Distribution Correction term (1.0 + 3.0/Alpha + 2.0/sqr(Alpha))

Term in mean particle mass equation, correction as,
    Mpm = 4.0/3.0 * M_PI * PBSM_constants::DICE * Radius*Radius*Radius;
    Removing Gamma Distribution Correction term (1.0 + 3.0/Alpha + 2.0/sqr(Alpha))

    Term in mean particle mass equation, correction as,
        Mpm = 4.0/3.0 * M_PI * PBSM_constants::DICE * Radius*Radius*Radius;
    Removing Gamma Distribution Correction term (1.0 + 3.0/Alpha + 2.0/sqr(Alpha))

    Term in mean particle mass equation, correction as,
        Mpm = 4.0/3.0 * M_PI * PBSM_constants::DICE * Radius*Radius*Radius;
 In evap module, /86400 to ra term for rs/ra in Penman-Monteith method, because rs is in d/m while ra is in s/m, correction as,

       evap[hh] = (delta(hru_t[hh])*Q*Global::Freq + (RHOa*Cp/
                  (lambda(hru_t[hh])*1e3)*(estar(hru_t[hh]) - hru_ea[hh])/(ra/86400)))/
                  (delta(hru_t[hh]) + gamma(Pa[hh], hru_t[hh])*(1.0 + rs[hh]/(ra/86400)))/
                  Global::Freq;
@jhs507 jhs507 requested a review from loganxingfang April 11, 2022 18:48
@jhs507 jhs507 marked this pull request as ready for review April 11, 2022 18:48
@jhs507
Copy link
Collaborator Author

jhs507 commented Apr 11, 2022

@lawfordp2017 You had a question about one of the changes earlier was that resolved?

@loganxingfang
Copy link
Collaborator

There should be one commit for the change on Line 29 in the Classinterception.cpp:

declvar("net_snow", TDim::NHRU, "Canopy unload ", "(kg/m^2*int)", &net_snow);

@jhs507
Copy link
Collaborator Author

jhs507 commented Apr 11, 2022

I have updated Classinterception

@jhs507
Copy link
Collaborator Author

jhs507 commented Apr 12, 2022

Are there more updates to implement or is this ready to be merged in?

@loganxingfang
Copy link
Collaborator

Are there more updates to implement or is this ready to be merged in?

These will be all updates for this time.

When my review comments above from yesterday are addressed, I think it is ready to be merged in.

@jhs507 jhs507 merged commit ae3deab into master Apr 12, 2022
lawfordp2017 added a commit that referenced this pull request Apr 29, 2022
Borland Updates March 21, 2022 (#265)
lawfordp2017 added a commit that referenced this pull request Apr 29, 2022
Borland Updates March 21, 2022 (#265)
@jhs507 jhs507 deleted the borland_updates_mar_21_22 branch May 26, 2022 16:37
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.

2 participants