-
Notifications
You must be signed in to change notification settings - Fork 1
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
staging -> production: DIST alerts with forest filters and encoded tiles titiler support #168
Conversation
…->alert to account for disturbance alerts
refactor integrated alerts algo and add dist alerts
Co-authored-by: Gary Tempus <[email protected]>
…-cache into titiler/encoded-alert-tiles
Titiler/encoded alert tiles
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## production #168 +/- ##
==============================================
+ Coverage 79.76% 79.86% +0.09%
==============================================
Files 57 61 +4
Lines 1898 2051 +153
==============================================
+ Hits 1514 1638 +124
- Misses 384 413 +29
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
Sweet refactor! Thanks for all the documentation, tests!
…r endpoints consistent with tile cache service
…m for dist/integrated alerts as we surface in api doc
Dev -> staging: DIST alerts forest filter
Bump dependencies for security and starlette changes
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.
Seems fine. Though I'm not totally sure if we need to push the dist alerts changes to production yet. Did you want to wait a bit and see if we have any more bug fixes or adjustments to endpoints, etc.? Whatever you think is best is fine.
fix tree cover loss year field type; update var name to make mypy happy
consistent and descriptive forest filter query params; detailed decod…
* 🐛 fix: Only replace implementation with `dynamic` Supports FLAG-1126 * ✅ test: add initial pytest suite * 🎨 refactor: create more declarative methods and tests * 📝 docs: add docstring to handler * 📝 docs: add implementation note regarding relative URLs
* 🐛 fix: Only replace implementation with `dynamic` Supports FLAG-1126 * ✅ test: add initial pytest suite * 🎨 refactor: create more declarative methods and tests * 📝 docs: add docstring to handler * 📝 docs: add implementation note regarding relative URLs (cherry picked from commit 8d79ee7)
switch from lambda to titiler tiling for integrated alerts
Dev to staging
We are hitting the edges of the whole-world bounding box for very north tiles, and need to add some rounding to deal with slight imprecision in the floating point computations.
Fix "Tile index out of bounds" error for very north tiles
Not needed, I merged the dist alert changes separately. |
No description provided.