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

Handle optional m and elev coordinates when transforming between geographic and cartesian geocentric coordinates #259

Open
navispatial opened this issue Jan 2, 2025 · 0 comments
Labels
enhancement New feature or request 🌐 geobase Related to the code package "geobase" refactoring

Comments

@navispatial
Copy link
Member

navispatial commented Jan 2, 2025

When optional m is available in source, it should be just copied to target.

Add omitElev parameter to conversions methods that result on geographic positions, this allow omitting elevation from the target:

  /// If [omitElev] is true, the optional elevation value is not calculated and
  /// it's set to null on the result geographic position. This can be useful if
  /// the elevation is near the ellipsoid surface and a non-zero elevation
  /// potentially resulted from the calculation is not wanted.

When converting from geographic to another geographic position between datums, and source do not have elevation then target should not either (even if calculation could generate one).

@navispatial navispatial added bug Something isn't working 🌐 geobase Related to the code package "geobase" labels Jan 2, 2025
@navispatial navispatial added this to the geobase v1.5 on dart 2.17 milestone Jan 2, 2025
@navispatial navispatial changed the title Preserve optional m coordinate when transforming between geographic and cartesian geocentric coordinates Handle optional m and elev coordinates when transforming between geographic and cartesian geocentric coordinates Jan 5, 2025
@navispatial navispatial added enhancement New feature or request refactoring and removed bug Something isn't working labels Jan 5, 2025
navispatial added a commit that referenced this issue Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request 🌐 geobase Related to the code package "geobase" refactoring
Projects
None yet
Development

No branches or pull requests

1 participant