search

opera-adt / COMPASS

COregistered Multi-temPorAl Sar Slc
Apache License 2.0
38 stars 18 forks source link

use `isce3::geocode::geocodeSlc` array mode instead of raster mode in `s1_geocode_slc` #127

Closed LiangJYu closed 1 year ago

LiangJYu commented 1 year ago

This PR replaces raster mode in s1_geocode_slc with array mode in isce3::geocode::geocodeSlc. Array mode includes compression options.

With compression on with default options, a CSLC product shrunk from 1.2G to 147M (this sounds too good to be true). Output was verified correct with visualization in ipython.

Needs this isce3 PR to be merged into develop.

LiangJYu commented 1 year ago

I benchmarked on my laptop (12 theads) and aurora (112 threads) and put the results here.

On my laptop, I was able to achieve parity between the array and raster mode by increasing lines/block per column. On aurora, I was able to reduce the difference from ~3x to ~2x by doing the same.

It's unclear at the moment why there's a difference in performance.