Skip to content
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

Merge PR #610 to master #614

Merged
merged 33 commits into from
Dec 19, 2024
Merged

Merge PR #610 to master #614

merged 33 commits into from
Dec 19, 2024

Conversation

danscales
Copy link
Collaborator

Merge Daniel's PR #610 to master/production - Don't use pixetl for geojson creation in resample script, should help us to make the zoom-14 tile cache for the global 10m JRC dataset

danscales and others added 30 commits November 14, 2024 15:04
…data

This is for testing of the pixetl. Will give it an official version name
if the testing is successful.
GTC-3015 Change to use new pixetl that parallelizes fetching of meta-data
@danscales danscales requested a review from dmannarino December 18, 2024 22:19
@codecov-commenter
Copy link

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

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.07%. Comparing base (0dbe8b6) to head (345e21a).

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

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #614   +/-   ##
=======================================
  Coverage   81.07%   81.07%           
=======================================
  Files         130      130           
  Lines        5876     5876           
=======================================
  Hits         4764     4764           
  Misses       1112     1112           
Flag Coverage Δ
unittests 81.07% <ø> (ø)

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.

Copy link
Member

@jterry64 jterry64 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, although curious why it's a hotfix instead of just merging develop if it's already in there?

@danscales danscales merged commit a6f21f9 into master Dec 19, 2024
1 of 2 checks passed
@danscales danscales deleted the merge_pr610_to_master branch December 19, 2024 23:36
@danscales
Copy link
Collaborator Author

Looks good, although curious why it's a hotfix instead of just merging develop if it's already in there?

I didn't want to merge in Daniel's change to the pixetl Dockerfile that is using an experimental pixetl.

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.

4 participants