Skip to content

BTS-2139: Accept z in GEO position definition #21829

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

Merged
merged 6 commits into from
Jul 1, 2025
Merged

Conversation

jbajic
Copy link
Contributor

@jbajic jbajic commented Jun 30, 2025

Scope & Purpose

Fix BTS-213: GeoJSON coordinate validation for Point geometries, now we accept lat,lng, z but still ignore z value during calculation.

The fix ensures GeoJSON Point geometries with optional Z coordinates are properly validated and processed according to the GeoJSON specification.

  • 💩 Bugfix
  • 🍕 New feature
  • 🔥 Performance improvement
  • 🔨 Refactoring/simplification

Checklist

  • Tests
    • Regression tests
    • C++ Unit tests
    • integration tests
    • resilience tests
  • 📖 CHANGELOG entry made
  • 📚 documentation written (release notes, API changes, ...)
  • Backports
    • Backport for 3.12.0: (Please link PR)
    • Backport for 3.11: (Please link PR)
    • Backport for 3.10: (Please link PR)

Related Information

(Please reference tickets / specification / other PRs etc)

@jbajic jbajic self-assigned this Jun 30, 2025
@cla-bot cla-bot bot added the cla-signed label Jun 30, 2025
@jbajic jbajic changed the title BTS-2139: Accept elevarion in GEO position definition BTS-2139: Accept z in GEO position definition Jun 30, 2025
Copy link
Member

@neunhoef neunhoef left a comment

Choose a reason for hiding this comment

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

LGTM. Let's inform Simran to make this behaviour clear in the documentation.

@goedderz goedderz merged commit d39098f into devel Jul 1, 2025
7 checks passed
@goedderz goedderz deleted the bug-fix/bts-2139 branch July 1, 2025 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants