insarlab / MintPy

Miami InSAR time-series software in Python
https://mintpy.readthedocs.io
Other
575 stars 252 forks source link

seq. closure phase: bring back spatial referencing for ARIA products #1063

Closed yunjunz closed 1 year ago

yunjunz commented 1 year ago

Description of proposed changes

This PR brings back the spatial referencing while calculating the sequential closure phase (which was removed in #922), for ARIA products only, to avoid the abnormal closure phase calculation result, as shown below. This is happening to ARIA products only, there is negligible impact on the interferogram stack processed with ISCE-2 from SLC (as @yjzhenglamarmota and I have tested before).

My guess is that the phase stitching applied during ARIA product preparation in ARIA-tools broke the temporal consistency of the native unwrapped phase (when used without spatial referencing). What do you think @sssangha @mgovorcin @dbekaert?

San Francisco Bay SenDT042 (downloaded from ARIA and processed with ARIA-tools)

Screenshot 2023-08-08 at 13 55 17 Screenshot 2023-08-08 at 13 55 07

Sierra Negra SenDT128 dataset (processed with isce2/topsStack)

Screenshot 2023-08-08 at 14 25 18 Screenshot 2023-08-08 at 14 25 27

Reminders

mgovorcin commented 1 year ago

@yunjunz did you run reference_point.py before this?

My guess is that GUNW ifgs might have different unwrapping seed point between pairs (this will especially change during stitching process) as coregistration is not done to the same reference image. The requirement would be to do spatial referencing before any further processing on them.

We do not do re-referencing of all ifgs to common reference point during prep_aria as that is done during reference_point step in the smallbaseline workflow. @sssangha might be good to add this to avoid potential misuse, what do you think?

yunjunz commented 1 year ago

@mgovorcin The first figure is without running reference_point.py.

Rethinking about this, I don't think the different seeding points or phase stitching would introduce this, because neither of them will change the wrapped phase value, which is what this figure is really using. The different reference images during pair-wise coregistration, as used in ARIA, sound more likely, as inconsistent range shifts are applied to both the resampling and SLC phase component, thus, could introduce this temporal inconsistency. Do you agree @hfattahi and @piyushrpt?