umr-lops / grdwindinversion

python library to compute wind speed from GRD SAR images
https://cerweb.ifremer.fr/datarmor/doc_sphinx/grdwindinversion/index.html
MIT License
9 stars 1 forks source link

Processing RS2 file seems buggy #35

Closed Skealz closed 1 year ago

Skealz commented 1 year ago

file : /home/datawork-cersat-public/cache/project/sarwing/data/RS2/L1/VV_VH/2022/248/RS2_OK141302_PK1242225_DK1208539_SCWA_20220905_212400_VV_VH_SGF

When processing this file I got not always the same output. Sometimes ncview bugs when displaying owiWindSpeed var, sometimes not. It seems that for RS2 files there sometimes are artifacts in the wind speed var.

rs2_1 rs2_2

Skealz commented 1 year ago

The picture using rs2_3 ncview when it successfully displays

vincelhx commented 1 year ago

here are my plots with my local .py

files : /home/datawork-cersat-public/cache/public/ftp/project/L2GRD/prod_v5/RS2_OK141302_PK1242225_DK1208539_SCWA_20220905_212400_VV_VH_SGF and /home/datawork-cersat-public/cache/public/ftp/project/L2GRD/prod_v5/RS2_OK141302_PK1242224_DK1208538_SCWA_20220905_212254_VV_VH_SGF

image image

Skealz commented 1 year ago

A buggy file that doesn't display using ncview /home/datawork-cersat-public/cache/project/sarwing/users/tcevaer/grdwindinversion/output_tests/RS2_OK141302_PK1242225_DK1208539_SCWA_20220905_212400_VV_VHSGF/rs2--owi-xx-20220905t212400-20220905t212516-____-_____.nc

agrouaze commented 1 year ago

after investigation with @vincelhx , the origin of the buggy fields for wind speed co and cross are related to xsarsea version. issue to open in this repo.

agrouaze commented 1 year ago

working version (nice wind speed fields) for @vincelhx using; xsar 0.9.7.dev164+g25db1e9 dev_0 xsarsea 0.3.dev122+ge1d9455 dev_0 outdated versions

agrouaze commented 1 year ago

issue solved with https://github.com/umr-lops/xsarsea/pull/55