Skip to content
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

Add prh30 to params file for fire tuning #206

Closed
billsacks opened this issue Dec 29, 2017 · 2 comments · Fixed by #1039
Closed

Add prh30 to params file for fire tuning #206

billsacks opened this issue Dec 29, 2017 · 2 comments · Fixed by #1039
Assignees
Labels
bfb bit-for-bit enhancement new capability or improved behavior of existing capability

Comments

@billsacks
Copy link
Member

From @ekluzek on May 23, 2017 17:57

Fang would like a new constant for tuning the CLM50 fire model. This constant only applies to Li2016 model and not the older version. So this acts a little bit different than other constants which are shared between both. So we may need to add a namelist read to just the newer version.

Here's the email from Fang which defines what to do...

Hi Dave and Erik,

 It would be better if Erik could help change (in CNFireLi2016Mod.F90)

arh30=1._r8-max(0.7_r8, min(1._r8,rh30_col(c)/90._r8))

to

arh30=1._r8-max(prh30, min(1._r8,rh30_col(c)/90._r8)) and define prh30=0.7_r8

In this way, we could tune the parameter (prh30) by just changing user_nl_clm if necessary, like ufuel...

Dave, low burned area in tropical deciduous tree region (casued by very high BTRAN2 and fuel load in r229) could be solved better if I set prh30 to 0._r8 or 0.4_r8, but I'm not sure it's performance in coupling case, CLM5's later version, or newer forcing data, so I didn't suggest change in prh30 value this time.

Thanks,

Fang

Copied from original issue: NCAR/CLM#8

@billsacks billsacks added the enhancement new capability or improved behavior of existing capability label Dec 29, 2017
@ekluzek
Copy link
Collaborator

ekluzek commented Jun 11, 2020

This will be moved to the params file for Li2016 in the hardcodedp branch coming to master.

@ekluzek ekluzek changed the title Add prh30 to namelist for fire tuning Add prh30 to params file for fire tuning Jun 11, 2020
@ekluzek
Copy link
Collaborator

ekluzek commented Jun 15, 2020

We are adding this on as a parameter on the paramsfile NetCDF file. I think the way that makes the most sense with this is to add the read for the parameter to the Li Fire base class, so that it will be read regardless of which Lit File Model is being used, and could be use for any future version as well. This would also be true for future fire variables that could be added in for the Li fire model series.

samsrabin pushed a commit to samsrabin/CTSM that referenced this issue May 3, 2024
@samsrabin samsrabin added simple bfb bit-for-bit labels Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bfb bit-for-bit enhancement new capability or improved behavior of existing capability
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants