search

Open-EO / openeo-geopyspark-driver

OpenEO driver for GeoPySpark (Geotrellis)
Apache License 2.0
26 stars 5 forks source link

Explicitly set all stac fields #918

Closed EmileSonneveld closed 1 month ago

EmileSonneveld commented 1 month ago

Tested on hrvpp-vi with http://stac.openeo.vito.be and http://stac-openeo-dev.vgt.vito.be (Automatic test needs to be enabled after merge: https://git.vito.be/projects/TPT/repos/os_creodias_openeo_k8s/commits/1bf7dbf0d2d16e17ef068ae5b1f0fc613f92b470 )

Tested on https://stac.terrascope.be/collections/sentinel-2-l2a

And of course the standard test suite

jdries commented 1 month ago

@EmileSonneveld why don't we just push on with the update of stac.openeo.vito.be? From what I understood, stac.terrascope.be has already been upgraded, so we're anyway forced to support the new behaviour already?

EmileSonneveld commented 1 month ago

stac.terrascope.be seems updated already indeed. So https://stac.terrascope.be/collections/sentinel-2-l2a is broken until this is merged. The code proposed here works with the new and the old VITO stac