Repository navigation
Fire raster holds the latest fire at or before dataYear - #141
Merged
Merged
Conversation
…275) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
eliotmcintire
added a commit
that referenced
this pull request
Oct 16, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014G18Mik3fH4z5M5CyMSUS6
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.
Stand ages inside old burns were wrong where a pixel burned twice: the later fire hid the earlier one, so for an earlier vegetation year the pixel kept its satellite age. The module now builds its fire map for the chosen vegetation year only.
Affects: stand age (and biomass re-estimated from it) inside reburned fire perimeters, for vegetation years earlier than the latest fire.
Depends-on: PredictiveEcology/LandR#275
Technical details
Defect: Biomass_borealDataPrep.R (the
prepInputsFireYear()call in.inputObjects, ~line 1810) rasterized all fires withfun = max, so a pixel burned in 1980 and 2015 held 2015 andreplaceAgeInFires(startTime = dataYear)skipped it fordataYear1990. The same raster is used for the young-biomass override (~line 1415, non-NA test only), which was missing the same pixels and is now right without other changes;earliestFireYearis untouched.Fix: pass
latestYear = P(sim)$dataYear;reqdPkgsfloor isLandR (>= 1.2.0.9057), the version LandR development (now 1.2.0.9056) will have once LandR#275 is merged and bumped. AfirePerimeterssupplied by another module is used as given.Test: a static check that the fire call passes
latestYear = P(sim)$dataYear(fails on development, passes on the branch); the raster behaviour is tested in LandR#275. Module suite (convertToPackage()+test_local(), scratch libraryphase4/libwith LandR from #275 installed as 1.2.0.9057): development 127 passed, 3 errors; branch 130 passed, 3 errors. The 3 errors are in test-estimateCoverModel.R (Copynot found) on both: the local harness does not attach reproducible, CI does, so they are not a module defect.🤖 Generated with Claude Code