Skip to content

Round MODIS projection origin coords#5099

Merged
dgboss merged 3 commits intomainfrom
bug/coarse-snow
Feb 4, 2026
Merged

Round MODIS projection origin coords#5099
dgboss merged 3 commits intomainfrom
bug/coarse-snow

Conversation

@dgboss
Copy link
Collaborator

@dgboss dgboss commented Feb 4, 2026

Snow coverage outputs are seeing some very coarse resolutions over northern Vancouver Island due to rounding/floating point math. This PR sets the MODIS projection origin to rounded integers as specified by NASA.

Test Links:

Landing Page
MoreCast
Percentile Calculator
C-Haines
FireCalc
FireCalc bookmark
Auto Spatial Advisory (ASA)
HFI Calculator
SFMS Insights
Fire Watch

@codecov
Copy link

codecov bot commented Feb 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.11%. Comparing base (24f571a) to head (60a5ea9).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5099   +/-   ##
=======================================
  Coverage   75.11%   75.11%           
=======================================
  Files         592      592           
  Lines       28713    28713           
  Branches     2107     2107           
=======================================
  Hits        21568    21568           
  Misses       6507     6507           
  Partials      638      638           

☔ View full report in Codecov by Sentry.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 4, 2026

@dgboss dgboss requested review from brettedw and conbrad February 4, 2026 03:19
@dgboss dgboss temporarily deployed to production February 4, 2026 18:02 Inactive
@dgboss dgboss merged commit 77926c2 into main Feb 4, 2026
32 checks passed
@dgboss dgboss deleted the bug/coarse-snow branch February 4, 2026 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants