ENH: STAC compliance - Githubissues

sertit / eoreader

Remote-sensing opensource python library reading optical and SAR sensors, loading and stacking bands, clouds, DEM and spectral indices in a sensor-agnostic way.

https://eoreader.readthedocs.io/en/latest/

Apache License 2.0

278 stars 22 forks source link

ENH: STAC compliance #29

Closed remi-braun closed 2 years ago

remi-braun commented 2 years ago

It could be nice to make eoreader compliant with STAC item format, by:

~~Changing the attributes names (i.e. extent to bbox, footprint to geometry)~~
~~Creating a to_stac() function for every product~~
[x] Create a stac attribute implementing all STAC relevant features but keep legacy EOReader functions (footprint, extent...)
[x] Changing band names
[x] Add informations to the bands (centrale wavelength, bandwidth, gsd, ...)

remi-braun commented 2 years ago

~~Add ESA resolution category (VHR1/2, HR1/2, OTHER)~~

Not relevant as SAR sensor can acquire in several resolution categories

remi-braun commented 2 years ago

[x] Use this changes to create a band object overloading bands and index (no more wild functions/enum)

remi-braun commented 2 years ago

Create namedtuples for STAC extensions in the STAC attribute:

[x] prod.proj for proj extension with projected data (i.e. call the projected bbox like prod.proj.bbox)
[x] prod.view for view extension
[x] prod.cloud_cover for cloud cover from the eo extension
[x] Enrich the band_names attribute (change name to bands) with everything here

remi-braun commented 2 years ago

[x] In the band object, add the output radiometric processing aimed, either reflectance or brilliance temperature.

remi-braun commented 2 years ago

[x] Rather than changing all the function names, it would be better to have a stac object as an attribute implementing all needed functions, i.e.:

# BBOX / extent
eoreader_bbox = prod.extent()
stac_bbox_wgs84 = prod.stac.bbox()
stac_bbox_utm = prod.stac.proj.bbox()  # == prod.extent()

# Export (needs pystac)
stac_item = prod.stac.create_item()

remi-braun commented 2 years ago

PR merged

remi-braun commented 2 years ago

⚠️ The view extension is partially filled. Are missing:

[x] view:off_nadir
[x] view:incidence_angle
[x] view:azimuth