isce2 icon indicating copy to clipboard operation
isce2 copied to clipboard
verified publisher icon isce-framework

bugfix in `cuDenseOffsets.py --full/out-geom` for `topsApp` product

Open yunjunz opened this issue 2 years ago • 1 comments

This PR fixes a bug while running cuDenseOffsets.py --full/out-geom on topsApp.py products by:

  • setting lat as the first geometry dataset, instead of hgt, because the former is shared between topsApp and topsStack products
  • add z from topsApp, in addition to hgt from topsStack, as the input geometry dataset

yunjunz avatar May 04 '23 13:05 yunjunz

Fixed the problem that I'm having with topsApp height products being processed through cuDenseOffsets.py.

bjmarfito avatar Jul 22 '24 08:07 bjmarfito

Hi @rtburns-jpl, can this PR be merged since I have coregistered SLC generated by topsApp and I will use PyCuAmpCor to generate pixel offsets? Thank you!

bjmarfito avatar Dec 06 '24 07:12 bjmarfito

Thanks for confirming the fix, will go ahead and merge it. Sorry for the wait!

rtburns-jpl avatar Dec 07 '24 00:12 rtburns-jpl

Thank you for merging!

bjmarfito avatar Dec 07 '24 02:12 bjmarfito