s1ard icon indicating copy to clipboard operation
s1ard copied to clipboard

Terrain Flattening results are currently underperforming

Open maawoo opened this issue 3 years ago • 5 comments

https://forum.step.esa.int/t/inferior-terrain-flattening-results-when-using-external-dem/37221

maawoo avatar Sep 01 '22 15:09 maawoo

Thanks for the great summary in this STEP issue. It was increasingly hard to follow the discussions around this topic. Let's hope they soon come up with a fix.

johntruckenbrodt avatar Sep 01 '22 15:09 johntruckenbrodt

Things are moving forward:

https://forum.step.esa.int/t/terrain-flattening-results-in-garbage-when-using-copernicus-30m-auto-download/33777/34

The reason that the TF result obtained using auto downloaded Copernicus 30m DEM is different from that obtained using external Copernicus 30m DEM file is that the DEM oversampling factor is computed differently. The oversampling factor is computed using the DEM resolution and the image pixel spacing. In the former case, the DEM resolution is given while in the latter case the DEM resolution is computed using Earth model. We have updated the oversampling factor calculation so that it is the same for both cases. The fix will be available in the next release.

johntruckenbrodt avatar Jan 27 '23 08:01 johntruckenbrodt

Finally! 🎉 Thanks for the update!

maawoo avatar Jan 27 '23 08:01 maawoo

https://github.com/SAR-ARD/S1_NRB/pull/78 should bring some improvement. To be revised once the new version is out.

johntruckenbrodt avatar Feb 08 '23 10:02 johntruckenbrodt

https://github.com/senbox-org/s1tbx/releases/tag/9.0.3

johntruckenbrodt avatar Mar 13 '23 07:03 johntruckenbrodt