add s1meta.bursts as DA without geopandas at this stage

umr-lops / xsar

Synthetic Aperture Radar (SAR) Level-1 GRD python mapper for efficient xarray/dask based processing

https://cyclobs.ifremer.fr/static/sarwing_datarmor/xsar/

MIT License

25 stars 8 forks source link

add s1meta.bursts as DA without geopandas at this stage #55

Closed agrouaze closed 2 years ago

agrouaze commented 2 years ago

address issue #54 objective: add s1meta.bursts

[x] add mappings of annotations
[x] add compound var
[x] validation

This PR doesn't contain the method in s1meta allowing to get polygons defined in atrack/xtrack because it requires that PR #46 (s1meta.geoloc) to be available.

agrouaze commented 2 years ago

In https://github.com/umr-lops/xsar/pull/55/commits/3e73a59b66691b899920d30df93629a2d1df48d2 I removed attribute lines_per_burst since it is accessible via the xarray.Dataset returned y s1meta.bursts and depending on selections it is potentially preferable to avoid hard coded attribute.

oarcher commented 2 years ago

Thanks @agrouaze , let's merge !