Skip to content

Commit

Permalink
Specify that mean_radius R1 is the mean radius of the ellipsoid semi-…
Browse files Browse the repository at this point in the history
…axes
  • Loading branch information
Mark Wieczorek committed Mar 25, 2024
1 parent 2cb926e commit f8a9138
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion boule/_ellipsoid.py
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,7 @@ def second_eccentricity(self):
@property
def mean_radius(self):
"""
The arithmetic mean radius of the ellipsoid [Moritz1988]_.
The arithmetic mean radius of the ellipsoid semi-axes [Moritz1988]_.
Definition: :math:`R_1 = (2a + b)/3`.
Units: :math:`m`.
"""
Expand Down
6 changes: 3 additions & 3 deletions boule/_sphere.py
Original file line number Diff line number Diff line change
Expand Up @@ -181,9 +181,9 @@ def first_eccentricity(self):
@property
def mean_radius(self):
"""
The mean radius of the sphere is by definition equal to its radius.
Added for compatibility with pymap3d.
Definition: :math:`R = R`.
The arithmetic mean radius of the ellipsoid semi-axes is equal to its
radius. Added for compatibility with pymap3d.
Definition: :math:`R_1 = R`.
Units: :math:`m`.
"""
return self.radius

Check warning on line 189 in boule/_sphere.py

View check run for this annotation

Codecov / codecov/patch

boule/_sphere.py#L189

Added line #L189 was not covered by tests
Expand Down
4 changes: 2 additions & 2 deletions boule/_triaxialellipsoid.py
Original file line number Diff line number Diff line change
Expand Up @@ -167,8 +167,8 @@ def _check_geocentric_grav_const(self, geocentric_grav_const, value):
@property
def mean_radius(self):
r"""
The arithmetic mean radius of the ellipsoid.
Definition: :math:`R = \dfrac{a + b + c}{3}`.
The arithmetic mean radius of the ellipsoid semi-axes.
Definition: :math:`R_1 = \dfrac{a + b + c}{3}`.
Units: :math:`m`.
"""
return (self.semimajor_axis + self.semimedium_axis + self.semiminor_axis) / 3
Expand Down

0 comments on commit f8a9138

Please sign in to comment.