Discussion: remove option to project to horizontal plane

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

Discussion: remove option to project to horizontal plane #365

Open adeane-ga opened 2 years ago

adeane-ga commented 2 years ago

Up for discussion...

Problem

The user has the option to project the LOS observation to the horizontal plane in the direction of the LOS vector.
This can be accessed from the user config file.
It is referred to as psuedo-horizontal projection.

Proposal Remove this capability because of the undetermined nature of LOS InSAR observations and the assumptions occurring with this projection could lead to mis-interpretation by users.

mcgarth commented 2 years ago

I don't see the sense in the proposal made here. We spent time adding it recently to provide flexibility to the user. As long as the feature is well documented (no doubt this can be improved!), IMHO it is up to the user to decide if and how they project the data, and accept the underlying assumptions.

adeane-ga commented 2 years ago

Yeah okay, if it doesn't make sense to remove it, then maybe the only action from this is to confirm the documentation and make sure it is well explained.