Repository navigation
Spin-up starts cohorts at initialB; widen young-cohort biomass check - #138
Merged
Merged
Conversation
…iomass limit Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CwcjqqK59FmTJscyi7xUqv
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CwcjqqK59FmTJscyi7xUqv
Collaborator
Author
|
This will change the estimated biomass upwards in the pixels that are "in fires", i.e., in |
eliotmcintire
added a commit
that referenced
this pull request
Oct 9, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CwcjqqK59FmTJscyi7xUqv
eliotmcintire
added a commit
that referenced
this pull request
Oct 9, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CwcjqqK59FmTJscyi7xUqv
achubaty
added a commit
to FOR-CAST/Biomass_borealDataPrep
that referenced
this pull request
Oct 19, 2026
Upstream 1.8.0 + PredictiveEcology#136, PredictiveEcology#138-PredictiveEcology#141 (LandR >= 1.2.0.9057); keeps SpaDES.project out of reqdPkgs and the fork NOTEs and test. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Young forests in recently burned areas get their starting biomass from a short growth simulation. That simulation started every tree species at zero biomass, so spruce barely grew while pioneer species took all the growth, and a sanity check on the result was too strict for fast-growing pioneers, stopping runs. This starts each species at the biomass a new cohort normally gets and loosens the check slightly.
Affects: starting biomass of young cohorts inside fire perimeters in every run (spruce higher, pioneers lower); runs that stopped on the check now continue.
Technical details
R/updateYoungBiomasses.R(spinUpPartial, was line ~164) setB = 0for every cohort, so growthcurve-1 species gained only the 1 g/m2/yr floor. Biomass_core gives a new cohortinitialB(default 10; LandR.initiateNewCohorts, cohorts.R:~330-335; the maxANPP branch only applies wheninitialBis NA). The module now has aninitialBparameter (same definition as Biomass_core, withparamCheckOtherMods(..., "warning")), passed tospinUpPartial, which uses it for the starting B (newspinUpStartCohorts()) and passes it to Biomass_core. IfinitialBis NA, cohorts start atmaxANPPfrom speciesEcoregion, as.initiateNewCohortsdoes with sumB = 0.Biomass_borealDataPrep.R:1467: the check now usesyoungBiomassLimit()=min(maxRawB, 1.2 * 2.8 * maxRawB / min(longevity / maxAgeHighQualityData)), still one scalar. 2026-10-09: ELF 3.2.3 had a 59-yr Lari_lar cohort 17% over the old limit, ELF 10.3.1 Pinu_con cohorts up to 2.9% over.New
tests/testthat/test-updateYoungBiomasses.R: the new tests error on development (functions absent). Full suite viaconvertToPackage+test_local: 135 pass, 0 fail on the branch (the metadata test lists the new parameter); on development 125 pass plus the new tests failing. The full spin-up with Biomass_core is not run in tests.🤖 Generated with Claude Code
https://claude.ai/code/session_01CwcjqqK59FmTJscyi7xUqv