Skip to content

Merge develop to master #693

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

Merged
merged 13 commits into from
May 5, 2025
Merged

Merge develop to master #693

merged 13 commits into from
May 5, 2025

Conversation

danscales
Copy link
Collaborator

Merge develop to master. Includes:

  • GTC-3276 - fix zoom-9 problem for integrated alerts on Pro
  • GTC-3268/wdpa aoi dm and subsequent fix

danscales and others added 12 commits April 29, 2025 23:34
The existing calculation was sometimes changing longitude 180 to -180
and vice versa. 180 and -180 are not equivalent, since a bounding box
always takes the shortest direction around, which is different for 180
(or 179.9999) and -180 (or -179.9999).

Change the calculation using to_4326() on cornerCoordinates (which is in
meters) to existing wgs84Extent, which is already exactly in degrees.
The resulting feature coordinates match the values from before the
switch from using pixetl.
GTC-3276 Change calculation of extent metadata using wgs84Extent
fix geostore query; useful error with missing latest wdpa version
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 38.46154% with 16 lines in your changes missing coverage. Please review.

Project coverage is 76.56%. Comparing base (e76272e) to head (7e3a1a8).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
app/crud/geostore.py 11.11% 8 Missing ⚠️
app/models/pydantic/datamart.py 52.94% 8 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #693      +/-   ##
==========================================
- Coverage   76.72%   76.56%   -0.16%     
==========================================
  Files         143      143              
  Lines        6681     6704      +23     
==========================================
+ Hits         5126     5133       +7     
- Misses       1555     1571      +16     
Flag Coverage Δ
unittests 76.56% <38.46%> (-0.16%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@danscales danscales merged commit 0c555c3 into master May 5, 2025
2 checks passed
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.

3 participants