Skip to content

Updating plugin keys for vertical transformation#4425

Open
gabriel-bolbotina wants to merge 3 commits into
masterfrom
fix/vertical-crs-keys
Open

Updating plugin keys for vertical transformation#4425
gabriel-bolbotina wants to merge 3 commits into
masterfrom
fix/vertical-crs-keys

Conversation

@gabriel-bolbotina
Copy link
Copy Markdown
Contributor

@gabriel-bolbotina gabriel-bolbotina commented Mar 26, 2026

skipElevationTransformation key is now deprecated in plugin, so the code was modified to support elevationTransformationEnabled.

If skipElevationTransformation was set to false, it means there was vertical transformation enabled, thus elevationTransformationEnabled will then be true and vice versa.

@gabriel-bolbotina gabriel-bolbotina changed the title Inital code for exchanging the keys Updating plugin keys for vertical transformation Mar 26, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 26, 2026

Coverage Report for CI Build 25729360743

Coverage decreased (-0.05%) to 59.833%

Details

  • Coverage decreased (-0.05%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 34 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

34 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
mm/app/position/positionkit.cpp 34 63.97%

Coverage Stats

Coverage Status
Relevant Lines: 14691
Covered Lines: 8790
Line Coverage: 59.83%
Coverage Strength: 103.7 hits per line

💛 - Coveralls

@github-actions
Copy link
Copy Markdown

📦 Build Artifacts Ready

OS Status Build Info Workflow run
macOS Build 📬 Mergin Maps 65761 dmg Expires: 24/06/2026 #6576
linux Build 📬 Mergin Maps 65871 x86_64 Expires: 24/06/2026 #6587
win64 Build 📬 Mergin Maps 57711 win64 Expires: 24/06/2026 #5771
Android Build 📬 Mergin Maps 788851 APK [arm64-v8a] Expires: 24/06/2026 #7888
📬 Mergin Maps 788851 APK [arm64-v8a] Google Play Store #7888
Android Build 📬 Mergin Maps 788811 APK [armeabi-v7a] Expires: 24/06/2026 #7888
📬 Mergin Maps 788811 APK [armeabi-v7a] Google Play Store #7888
iOS Build 📬 Build number: 26.03.882711 #8827

Copy link
Copy Markdown
Contributor

@Withalion Withalion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a question, why do we want to support both keys? Plugin never got released with the support for this functionality and we just enabled it in few project for very early adopters.

Besides that it looks alright.

@github-actions
Copy link
Copy Markdown

📦 Build Artifacts Ready

OS Status Build Info Workflow run
macOS Build 📬 Mergin Maps 66031 dmg Expires: 29/06/2026 #6603
linux Build 📬 Mergin Maps 66141 x86_64 Expires: 29/06/2026 #6614
win64 Build 📬 Mergin Maps 57981 win64 Expires: 29/06/2026 #5798
Android Build 📬 Mergin Maps 791511 APK [armeabi-v7a] Expires: 29/06/2026 #7915
📬 Mergin Maps 791511 APK [armeabi-v7a] Google Play Store #7915
Android Build 📬 Mergin Maps 791551 APK [arm64-v8a] Expires: 29/06/2026 #7915
📬 Mergin Maps 791551 APK [arm64-v8a] Google Play Store #7915
iOS Build 📬 Build number: 26.03.885411 #8854

@github-actions
Copy link
Copy Markdown

📦 Build Artifacts Ready

OS Status Build Info Workflow run
macOS Build 📬 Mergin Maps 68281 dmg Expires: 10/08/2026 #6828
linux Build 📬 Mergin Maps 68461 x86_64 Expires: 10/08/2026 #6846
win64 Build 📬 Mergin Maps 60211 win64 Expires: 10/08/2026 #6021
Android Build 📬 Mergin Maps 813851 APK [arm64-v8a] Expires: 10/08/2026 #8138
📬 Mergin Maps 813851 APK [arm64-v8a] Google Play Store #8138
Android Build 📬 Mergin Maps 813811 APK [armeabi-v7a] Expires: 10/08/2026 #8138
📬 Mergin Maps 813811 APK [armeabi-v7a] Google Play Store #8138
iOS Build Build failed or not found. #9080

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adjust vertical CRS transformation code according to recent QGIS-plugin changes

3 participants