Merge dist to prod #183
Merged
Merge dist to prod #183
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…->alert to account for disturbance alerts
refactor integrated alerts algo and add dist alerts
…-cache into titiler/encoded-alert-tiles
Titiler/encoded alert tiles
…r endpoints consistent with tile cache service
…m for dist/integrated alerts as we surface in api doc
make titiler tile endpoints consistent with tile cache service
Gtc 3025/filter alerts by forest
Dev -> staging: DIST alerts forest filter
update dataset name; allow test datasets
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 (cherry picked from commit 8d79ee7)
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.
|
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 #183 +/- ##
==============================================
+ 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. |
jterry64
approved these changes
Dec 12, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Merge all of Solomon's changes to support using titiler for DIST alerts from staging to production. This includes his changes to make the alerts code more generic, so it can be shared between integrated alerts and dist alerts.
This merge explicitly does not include the changes in staging to make integrated alerts tile cache calls use titiler by default, since we are still testing that with Flagship in staging.