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

Small flattening ellipsoids #197

Open
wants to merge 27 commits into
base: main
Choose a base branch
from

Conversation

MarkWieczorek
Copy link
Contributor

@MarkWieczorek MarkWieczorek commented May 3, 2024

This PR extends the Ellipsoid class to allow the use of flattenings that are close to, and including, zero. The textbook equations used in the previous version ran into numerical instabilities when the flattening was lower than about $10^{-7}$. This was a result of either divide by 0 errors when the linear eccentricity approached zero, or a result of square roots of negative numbers (like -1.e-30) when computing the angle $\beta^{\prime}$.

This PR uses Taylor series expansions about $f=0$ that do not suffer from these numerical problems. The equations and their applicability are presented in #194. In general, for each routine that is concerned, the flattening is checked, and if it is smaller than a given value, the small flattening equations are used instead of the textbook equations.

The following routines have been modified:

  • Ellipsoid.reference_normal_gravity_potential
  • Ellipsoid.normal_gravitational_potential
  • Ellipsoid.normal_gravity_potential
  • Ellipsoid.normal_gravity
  • Ellipsoid.gravity_equator
  • Ellipsoid.gravity_pole
  • Ellipsoid.geodetic_to_ellipsoidal_harmonic

The web documentation for the normal gravity routines has been cleaned up a little to better explain the difference between an ellipsoid with zero flattening and a sphere.

Note that the code in Ellipsoid.geodetic_to_ellipsoidal_harmonic is largely repeated in Ellipsoid.normal_gravity. It might be preferable to simply call geodetic_to_ellipsoidal_harmonic within normal_gravity for clarity and brevity.

Note also that this PR is based on this other (not yet merged) PR: #187. It it probably better to address #187 first, given the number of changes made in both of these.

Relevant issues/PRs:
Relevant to #194

Mark Wieczorek and others added 26 commits April 8, 2024 14:48
@MarkWieczorek MarkWieczorek changed the title WIP: Small flattening ellipsoids Small flattening ellipsoids May 3, 2024
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.

None yet

1 participant