-
Notifications
You must be signed in to change notification settings - Fork 136
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
Update fire_emissions_factors in hist_trop_strat_vbsfire_cam6 usecase #1148
Comments
@fvitt - Do you agree that this change is appropriate? Does @cecilehannay need this for her upcoming runs? |
Adding @tilmes @lkemmons to the discussion. Note, that we talked about this over email with Louisa, Simone, Francis, Will, and Sam R. in the context of CTSM in ESCOMP/CTSM#2734 on Aug/29th. What I then realized is that CAM needs this same update. The difference in this file from the previous one, is just in Comp_EF and Fang Li @lifang0209 talked about the reason for the changes were that some PFT's had the emission factors incorrectly supplied. For example, Fang found that
@cacraigucar unless this use case is needed for @cecilehannay upcoming simulations -- this won't be required. Since, this use-case is really specialized I find that doubtful. |
Issue Type
Other (please describe below)
Issue Description
Update the fire_emissions_factors filename in the hist_trop_strat_vbsfire_cam6 to the one being brought into CTSM in ctsm5.3.0.
The change in CTSM is here:
https://github.com/ESCOMP/CTSM/pull/2500/files#diff-efe0e044594e12ba9835f0871f3ffd821575589c8668db139f8bffa25718127a
See:
ESCOMP/CTSM#2734
Will this change answers?
Yes
Will you be implementing this yourself?
No
The text was updated successfully, but these errors were encountered: