Skip to content

defaultFireYears() starts in 1986 - #158

Merged
eliotmcintire merged 2 commits into
developmentfrom
fire-years-start-1986
Oct 15, 2026
Merged

eliotmcintire merged 2 commits into
developmentfrom
fire-years-start-1986

Conversation

@eliotmcintire

@eliotmcintire eliotmcintire commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

The default fire years now start in 1986, one year after the first vegetation year, because each year's fires are now fitted against vegetation from an earlier year. Fires of 1985 can no longer be paired with any vegetation.

Affects: the default fire years of fireSense_dataPrepFit and fireSense_ELFs (ELF fire counts for merging now exclude 1985).

Technical details

Defect: defaultFireYears() (R/defaultFireYears.R:15) returned 1985L:climateData::latestHistoricalYear(). SCANFI vegetation of year Y already shows year Y's burns, so fireSense_dataPrepFit#73 pairs each fire year with the latest data year strictly before it, and 1985 has none. Fix: it returns 1986L:...; roxygen and man/ updated. Other uses (R/fireSenseCloudParameterStatus.R:120, plotELFs()) call the function and need no change; the 1985 in ledger file names and dataYears examples is unrelated. test-defaultFireYears.R now expects 1986 (fails on the old code: 2 failures). R CMD check (--no-manual, suggested packages PtProcess, spatstat, spelling not installed so _R_CHECK_FORCE_SUGGESTS_=false, scratch library phase4/lib): 0 errors, 0 warnings, 0 notes; tests 0 failed, 1979 passed, 10 skipped. Base (development): 0 failed, 1994 passed, 2 skipped (devtools::test).

🤖 Generated with Claude Code

…tation from an earlier year

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@eliotmcintire eliotmcintire added bug Something isn't working breaking change Existing code or results need changes to keep working labels Oct 15, 2026
@codecov

codecov Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.13%. Comparing base (7cc9645) to head (19e0b15).

Additional details and impacted files
@@             Coverage Diff              @@
##           development     #158   +/-   ##
============================================
  Coverage        89.13%   89.13%           
============================================
  Files               50       50           
  Lines             4398     4398           
============================================
  Hits              3920     3920           
  Misses             478      478           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@eliotmcintire
eliotmcintire merged commit ace00a0 into development Oct 15, 2026
16 checks passed
@eliotmcintire
eliotmcintire deleted the fire-years-start-1986 branch October 15, 2026 23:46
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change Existing code or results need changes to keep working bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant