Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Discussion: remove option to project to horizontal plane #365

Open
adeane-ga opened this issue Oct 18, 2021 · 2 comments
Open

Discussion: remove option to project to horizontal plane #365

adeane-ga opened this issue Oct 18, 2021 · 2 comments

Comments

@adeane-ga
Copy link
Contributor

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
Copy link
Contributor

mcgarth commented Oct 29, 2021

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
Copy link
Contributor Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants