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

Add option for geodetic/geopotential altitude input to the atmosphere model. #725

Closed
1 of 4 tasks
robfalck opened this issue Mar 15, 2022 · 0 comments · Fixed by #735
Closed
1 of 4 tasks

Add option for geodetic/geopotential altitude input to the atmosphere model. #725

robfalck opened this issue Mar 15, 2022 · 0 comments · Fixed by #735
Assignees

Comments

@robfalck
Copy link
Contributor

robfalck commented Mar 15, 2022

Issue Type

  • Bug
  • Enhancement
  • Docs
  • Miscellaneous

Description

The atmosphere model in Dymos was originally intended to be a low-fidelity model intended to support some example cases.
As our userbase has grown, a number of projects are using this model and requesting higher fidelity.

The 1976 standard atmosphere implementation should correct the geodetic altitude input and perform the interpolation using the geopotential altitude. The Dymos implementation currently assumes that the input altitude is already the geopotential altitude.

To address this, we will add an option to the atmosphere component:

h_def

Which will take one of two options: geopotential and geodetic.

The default will be geopotential and will provide the current behavior.
If h_def is geodetic, then the incoming geodetic altitude will first be converted to the geopotential altitude using Equation 19 from Reference 2 (Equation 25 from Reference 1)

z = R0 * h / (R0 + h)

Where Re is 6356.766 km, z is the geopotential altitude, and h is the geodetic altitude.

References

  1. A Discussion of Various Measures of Altitude
  2. US Standard Atmosphere 1976

Example

N/A

Environment

N/A

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 a pull request may close this issue.

1 participant