-
Notifications
You must be signed in to change notification settings - Fork 29
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
Homogenization 2.0 #254
Homogenization 2.0 #254
Conversation
- Update vignettes - Save full model outputs for Biomee (including site level) - Move BiomeE integration test - Change QMD -> QMD12 - Remove unused parameters for BiomeE (outputhourly, outputdaily, dist_frequency, do_reset_veg)
@fabern I haven't updated the documentation for P-model (point 4 in the old pull-request). I don't know p-model at all. I also completely re-written data_format.Rmd. To avoid back and forth, I propose that you just commit the changes you wish directly in this branch (253). |
Thanks @marcadella, most of my suggested changes to the order of the docs were already done by you in commit (499b2c2), I now made small, last adjustments in (63f29b3). This is ok to cover point 4. |
…rection of psoil_sl leading to negative values.
- Fix post mortality metrics never set in annual_diagnostics - Fix annualfixedN never updated
Ag_l exploding causes Psyn, Gpp, plabl%c12, C_push, C_growth, dBSW, cc%DBH and oldcc%crownarea to be NaN Note that if this removes the segfault, we have not treated the root cause of the issue causing Ag_l to explode in the first place.
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.
Looks good from a quick glance. Thanks!
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.
ok
See issue #253