SFMS: Filter SFMS interpolation stations by site type#5230
Conversation
|
Part of the acceptance criteria is also to only use 'Active' stations for SFMS. Check out |
|
Could you also potentially run some tests locally and see if we're now getting the same stations as current SFMS? |
Thought we already had that in |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5230 +/- ##
=======================================
Coverage 69.20% 69.20%
=======================================
Files 393 393
Lines 16262 16265 +3
Branches 1822 1822
=======================================
+ Hits 11254 11257 +3
Misses 4471 4471
Partials 537 537 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
What do you think about adding a comment stating that the |
|



Excludes stations with site types not suitable for spatial interpolation (
HUB_STN, WXSTN_GOES, WXSTN_MB, WXSTN_TEL, WXSTN_CELL, WXSTN_UHF, EXTERNALare the valid types as well asACTIVEstatus).The filter is applied in
sfms_daily_actuals_mapperin Python rather than via the RSQL query on/dailies/rsql, because the existing codebase already documents thatthe API does not reliably honour RSQL filters on nested stationData properties (the same issue affects stationStatus filtering).
Closes #5206
Test Links:
Landing Page
MoreCast
Percentile Calculator
C-Haines
FireCalc
FireCalc bookmark
Auto Spatial Advisory (ASA)
HFI Calculator
SFMS Insights
Fire Watch