-
Notifications
You must be signed in to change notification settings - Fork 76
Custom geoid support #4238
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
Custom geoid support #4238
Conversation
Pull Request Test Coverage Report for Build 19823024385Details
💛 - Coveralls |
7fad8a4 to
1d770ff
Compare
tomasMizera
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job 🔝
|
@Withalion shouldn't we set Also, what happens if a vertical grid is missing in |
|
fixes #4237
By default the app uses EGM96 geoid model to recalculate the elevation. Sometimes there is a need for more precise model, which is local to some area. With the work done in MerginMaps/qgis-plugin#841 admins can setup custom geoid model. This PR changes the mobile app behaviour to use the custom model if it's setup and shows user the current model in use.
For local geoid models, if current position is outside the model area we should show
N/Avalues for Altitude and Geoid Separation. The code for this behaviour might be missing in some providers,I'm waiting forit will be implemented indev/geoid-supportbe mergeddev/geoid-support