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

Add estimated elevation field in sarvey_export output #32

Open
mahmud1 opened this issue Jan 16, 2025 · 3 comments
Open

Add estimated elevation field in sarvey_export output #32

mahmud1 opened this issue Jan 16, 2025 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@mahmud1
Copy link
Member

mahmud1 commented Jan 16, 2025

It would be helpful to include an estimated elevation field in sarvey_export. This field would provide an estimate of the elevation based on dem+dem_error+offset. Offset is geoid height and can be incorporated as an additional argument.
What do you think @Andreas-Piter?

@mahmud1 mahmud1 added the enhancement New feature or request label Jan 16, 2025
@Andreas-Piter
Copy link
Member

Yes, sounds good. The correct elevation should be computed anyway (#16 ), and I agree, we could add offset as argument.

@mahmud1 mahmud1 self-assigned this Jan 17, 2025
@yurioliveira-ctec
Copy link

yurioliveira-ctec commented Feb 4, 2025

Dear @Andreas-Piter,

I hope you are doing well.

I was looking into the issue of elevation estimation and had a question: Is the DEM error generated by SARvey in the LOS direction? If so, would it be necessary to convert it to altitude in order to sum it with the other components (DEM and offset)?

@Andreas-Piter
Copy link
Member

Dear @yurioliveira-ctec,
The DEM correction estimated in SARvey is with respect to a reference point. In order to achieve the actual height of the scatterer, an offset w.r.t. the geoid and the DEM height needs to be added.
If you require the height in the output shapefile, you can manually compute the value as described above by Mahmud above. We will add this soon to the software.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants