-
Notifications
You must be signed in to change notification settings - Fork 341
Fire update May 2025 #3204
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
Fire update May 2025 #3204
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pretty simple and mostly looks good. Just some changes to request, which I can handle.
-
defo_fire_precip_thresh_betalso needs to be changed, just for CRU-JRA. New value 3.0. - Change
cropfire_a1to 0.32, even if we get rid of the 10% crop fraction threshold - Change values on parameter file to match those in
/glade/u/home/fangli/ctsm60_params.fire.crujra0530.nc:
(1) rswf_max(3)=0.7d00
(2) rswf_min(3)=0.35d00
(3, 4) multiplying fsr_pft by 5 and dividing fd_pft by 5, based on FireAtlas and more reasonable, does not affect the results
|
Blocked by #3188. |
|
This is unblocked; now waiting on @lifang0209 to change the code to avoid the crop fraction threshold now that #3188 is merged. |
|
@samsrabin Hi Sam, (2) remove “.and. cropf_col(c) * col%wtgcell(c) > 0.1_r8 ” from the condition statement |
|
Great, thanks! Will do. |
|
Blocked while we wait for the 5.4 update, since this depends on new surface datasets. |
|
We have the 5.4 branch. Can / should this PR get merged to the 5.4 branch so we can start evaluating fire results in coupled model tests? |
|
Yes, I think it should be merged. |
|
@wwieder @lifang0209 This is ready for testing! You can see the script I used to apply the paramfile changes here: |
No longer needed now that ESCOMP#3188 is resolved.
|
Note that this will show a lot of diffs until the |
Set init_interp_fill_missing_urban_with_hd = .true. always We will change this when we complete the ctsm5.4 release. At that time we want the flag = .true. for IHist cases.
|
|
I suggest this come in after the 5.4 tag is made, giving us a bit more time to generate initial conditions and evaluate historical fire results on top of the PPE115 parameter file that's being used in NCAR/LMWG_dev#116 |
|
@wwieder Hi Will, if the 5.4 tag uses CTSM5.4 surface and land use data, it‘s better to include the fire scheme updates. The updates include recalibration based on new abm (crop peak month) and land-use-based deforestation rate. |
|
@ekluzek Should this be tag |
Update the ctsm5.4 alpha branch to ctsm5.3.071 PR ESCOMP#3464
d07cc09 to
3123c32
Compare
0d32413 to
6c08359
Compare
Description of changes
For now, also makes it so that crop fires don't happen if cropland is < 10% of the gridcell. However, this was only a kludge to avoid some mathematical weirdness with interpolation of agricultural peak burning month using average. This will be replaced by a more appropriate interpolation method with #3188, after which maybe remove 10% cropland threshold. However, that will make cropland burned area 10-30 Mha worse.
Specific notes
Contributors other than yourself, if any: None
CTSM Issues Fixed: None
Are answers expected to change (and if so in what way)? Yes. @lifang0209, can you please add a comment explaining the results before and after?
Any User Interface Changes (namelist or namelist defaults changes)?
Namelist defaults:
defo_fire_precip_thresh_betfor CRU-JRAcli_scalefor CRU-JRA and GSWP3Parameter file:
rswf_max(3)rswf_min(3)fsr_pftandfd_pft, although these changes cancel each other outDoes this create a need to change or add documentation? Did you do so? No
Testing performed, if any:
@lifang0209, please add a comment explaining scientific testing.
Before merging:
rimportnew parameter file.aux_clm, expecting diffs only inli202[14]*fire methods. (2021 because of paramfile changes torswf_min/max; 2024 due to those changes and others.)