Geoid separation support for external providers#4176
Merged
Withalion merged 2 commits intodev/geoid-supportfrom Nov 19, 2025
Merged
Geoid separation support for external providers#4176Withalion merged 2 commits intodev/geoid-supportfrom
Withalion merged 2 commits intodev/geoid-supportfrom
Conversation
Pull Request Test Coverage Report for Build 18729738405Details
💛 - Coveralls |
(cherry picked from commit 76b65ba)
wonder-sk
reviewed
Oct 30, 2025
Contributor
wonder-sk
left a comment
wonder-sk
left a comment
There was a problem hiding this comment.
Looks good - and testing with external GNSS confirms it works.
One thing worth fixing: with internal GNSS, the GPS info panel shows geoid separation being "0.00 m" which could be confusing - it would be better to report it as N/A. Or maybe we could add calculation of geoid separation for internal GNSS while at this:
ct=QgsCoordinateTransform(
QgsCoordinateReferenceSystem("EPSG:4979"), # WGS84 + ellipsoid height
QgsCoordinateReferenceSystem("EPSG:9707"), # WGS84 + EGM96 geoid height
QgsCoordinateTransformContext()) # TODO: use project's context
# this is ellipsoid height of 200 m converted to geoid height - at lat 49°, lon 18°
print(ct.transform(QgsVector3D(18,49,200)).z())(we just need to verify we have egm96 geoid shipped with PROJ - on my system it is in file egm96_15.gtx - cca 4 MB)
Contributor
|
Let's keep this PR as it is and I'll create a new one to resolve the issue with internal providers. It seems better to me as we will be building on top of the work done here and also in #4210. |
Withalion
changed the title
Withalion
approved these changes
Nov 19, 2025
Withalion
pushed a commit
that referenced
this pull request
Dec 12, 2025
Withalion
added a commit
that referenced
this pull request
Feb 9, 2026
* Geoid separation support for external providers (#4176) * Recalculate ellipsoid elevation to orthometric (#4210) * Enhance position altitude processing Add EGM96_15 geoid model, which recalculates ellipsoid altitudes returned by position providers. Expose this information in GPS information panel. * Fix broken builds * Fix formatting * Add geoid info for iOS * Refactor PositionKit to singleton from context property * Patch ios internal positioning provider Provider returns now WGS84 ellipsoidal height on iOS * Fix elevation transform & android workaround Create new 3D transform utils function. Fix coordinate order passing. Rework android 15+ Qt positioning workaround to VCPKG patch. * Clean up & format code * Add patch TODO * Fix some review issues * Refactor code to use existing QgsCoordinateTransformContext (#4228) * Geoid separation support for internal providers (#4216) * Squashed commit of the following: commit 27b19a4 Author: Matej Bagar <matej.bagar@lutraconsulting.co.uk> Date: Mon Nov 24 15:22:46 2025 +0200 Add mock location detection for android position provider * Change default value for elevation_diff * Custom geoid support (#4238) * Fixed plural translation forms (#4183) * Fixed android build to sync projects (#4243) * Refactor support for orthometric heights on iOS * Subtract antenna height from altitude * Fix formatting & vcpkg bug * Fix ios patch * Fix ios patch v2 * Fix iOS error & constness * Fix iOS error v3 * Add ellipsoidal elevation expr variable * Change elevation calculation * Increase click area for elevation info button * Use default transform for internal providers * Use default geoid model for android fused provider * Add vertical CRS transform pass through for mock providers * Add vertical CRS transfrom pass through for bluetooth provider * Change android fused provider to default on android * Fix GPS panel geoid model name info * Change behaviour for internal provider on desktops * Use EGM96 geoid model for simulated provider * Fix iOS build * Small refactor * Set internal providers to use mixture of vertical CRS * Fix review issues * Refactor providers to use PositionTransformer * Add PositionTransform reload on project load * Add unit tests for PositionTransformer * Rename project setting name * Remove commented code, minor typo fixes, simulated provider cleanup * Fix tests & formatting * Fix tests * Add test debug logs for CI * Fix wrong test initialization for bluetoothless devices * Revert "Fix wrong test initialization for bluetoothless devices" This reverts commit 1541538. * Refactor PositionTransformer creation & clear QgsProject refs in PositionKit * Remove InputCoordinateTransformer * Fix review issues * Fix function typo --------- Co-authored-by: Tomas Mizera <tomas.mizera@lutraconsulting.co.uk> Co-authored-by: Kaustuv Pokharel <85729205+kaustuvpokharel@users.noreply.github.com> Co-authored-by: Gabriel Bolbotina <80618569+gabriel-bolbotina@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Just a quick demo for geoid separation
Resolves #2725