search

GeoscienceAustralia / PyRate

A Python tool for estimating velocity and time-series from Interferometric Synthetic Aperture Radar (InSAR) data.
https://geoscienceaustralia.github.io/PyRate/
Apache License 2.0
199 stars 69 forks source link

Reduce cropping options #364

Open adeane-ga opened 2 years ago

adeane-ga commented 2 years ago

Simple Change to clarify usage Currently PyRate has 4 options to crop, would be better if we reduce to 2 because the others are redundant and confusing.

Current Options: # ifgcropopt: 1 = minimum extent 2 = maximum extent 3 = crop 4 = no cropping

Proposed Change: # ifgcropopt: 1 = crop 2 = no cropping

mcgarth commented 2 years ago

I agree with the proposed change, but I would suggest a binary-style switch:

# ifgcropopt: 0 = do not crop; 1 = crop

adeane-ga commented 2 years ago

I agree with the proposed change, but I would suggest a binary-style switch:

# ifgcropopt: 0 = do not crop; 1 = crop

Yeah I agree, makes sense.