-
Notifications
You must be signed in to change notification settings - Fork 10
SFMS Insights: Fire Weather Raster Visualization #4949
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
web/src/features/sfmsInsights/components/map/layerManager.test.ts
Outdated
Show resolved
Hide resolved
| } | ||
|
|
||
| // Backward compatibility | ||
| export const fwiColourExpression = () => getFireWeatherColourExpression('fwi') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this is used anywhere
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed in: a732899
|
| warped = gdal.Warp( | ||
| "", # Empty string creates in-memory dataset | ||
| src_ds, | ||
| format="MEM", | ||
| dstSRS=target_srs, | ||
| resampleAlg=resample_alg.value, | ||
| ) | ||
|
|
||
| if warped is None: | ||
| raise RuntimeError(f"Failed to warp raster to {target_srs}") | ||
|
|
||
| # Translate to COG format with auto-generated overviews | ||
| result = gdal.Translate( | ||
| output_path, | ||
| warped, | ||
| format="COG", | ||
| creationOptions=[ | ||
| f"COMPRESS={compression}", | ||
| "BIGTIFF=IF_SAFER", | ||
| "OVERVIEWS=IGNORE_EXISTING", # Always create new overviews | ||
| ], | ||
| ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking at this: https://gdal.org/en/stable/drivers/raster/cog.html#reprojection-related-creation-options, I'm not sure we actually need the step of Warping first, it might all be able to be done in the COG creation options. Either way is fine I think, it would just remove a step potentially. Otherwise I think the creation and batch creation scripts look good!
brettedw
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is awesome, great work!
My only thought is that we should follow it up soon with an 'About this data' button or something similar, since the rasters we're showing here are different from the ones in WF1. It would reduce confusion for anyone checking it out
dgboss
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing work and test coverage!



Adds new SFMS Insights page with interactive map visualization of fire weather raster data (GeoTIFF) from BC government object store.
Features:
wps_tools.generate_sfms_cogscript and launch config to selectively generate COGs for the frontend (currently ran for June 27, 2025)Technical Implementation:
Closes #4169
Closes #4170
Closes #4182
Closes #4161
Closes #4171
Closes #4183
Closes #4959
Test Links:
Landing Page
MoreCast
Percentile Calculator
C-Haines
FireCalc
FireCalc bookmark
Auto Spatial Advisory (ASA)
HFI Calculator
SFMS Insights
Fire Watch