-
Notifications
You must be signed in to change notification settings - Fork 92
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
Mass balance error in TRS #1246
Comments
Hi! The TRS parameters look good to me. The mass balance error is likely related to something else. It appears this happens in the first time step? Did you initialize the model with observed stand structure? I recommend you try running the exact same simulation with the TRS turned off to confirm if this is a TRS problem or something else. |
Hello! I've tried again by re-creating 3 parameter files; the only changes made to each file from the default is the following,
and
for the first, second and third file respectively. I then ran the model 3 times, each time changing the parameter file the script directs to. The first (fates_regeneration_model = 1 ) and third (fates_regeneration_model = 3 ) managed to run the simulations successfully, however I still encountered the same issue with the second (fates_regeneration_model = 2 ) with the mass balance error. For the initialization inventory, I used data from one census of the 4 ha ForestGEO plot in Singapore (Bukit Timah), with pft = 0 for all the trees in the .css file. Thank you! |
Ok, @ongalicia. Which version of the FATES code are you using (which api tag number?)? @glemieux, I see you linked this issue to another issue. Is this something you're familiar with? I haven't used the TRS for a while, but perhaps this is either a newer bug that has something to do with TRS compatibility with a newer api or an old bug that is only just now being discovered? |
I am using the |
Apologies @adamhb, the linked issue is one I was creating as part of a github demonstration. You can ignore it. This was discovered during our FATES tutorial that we held last week. Your question about it being a newer bug is a good one. Upon reflection, we don't have a test routine in our standard regression suite to exercise the TRS modes. This is something I'll address in a different issue. UPDATE: created issue #1251 |
@glemieux, thanks for starting #1251. @ongalicia, the problem could be that a bug was introduced in a more recent version of FATES that conflicts with the TRS. Hopefully, work on the new issue #1251 that @glemieux started will help solve this. |
@adamhb I tried to run FATES by editing the default parameter file for 3 PFTs (default values from "Broadleaf evergreen tropical tree" and edited selected values). I followed the parameter values in Handbury-Brown et al. (2022) (https://doi.org/10.1111/nph.18059) for LD-DI, ST-DI and ST-DT (edited parameter values as below). The fates_regeneration_model was set to be = 2 to be the "Tree Recruitment Scheme (Hanbury-Brown et al., 2022)".
The text was updated successfully, but these errors were encountered: