opendatacube / eo-datasets

Easily write, validate and convert EO datasets and metadata.
Apache License 2.0
49 stars 26 forks source link

Include DEA's new offshore territories tag #335

Closed jeremyh closed 4 months ago

jeremyh commented 4 months ago

... and add a test case of a modern wagl output.

The added test data found a bug unrelated to the offshore territories: when the first measurement has no explicit CRS it fails with a reference error.

I've also unwisely included a few housekeeping commits here.

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 83.33333% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 84.44%. Comparing base (bd57906) to head (e080e1c). Report is 6 commits behind head on develop.

Files Patch % Lines
eodatasets3/prepare/nasa_c_m_mcd43a1_6_prepare.py 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #335 +/- ## =========================================== + Coverage 84.37% 84.44% +0.06% =========================================== Files 26 26 Lines 4506 4506 =========================================== + Hits 3802 3805 +3 + Misses 704 701 -3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

jeremyh commented 4 months ago

(The Docker Push action is failing for an unrelated reason — missing dockerhub credentials. I'm not an repo admin so can't view/setup credentials https://github.com/opendatacube/eo-datasets/issues/336)