Skip to content

Sea Level Density overhaul -> Requires OpenMDAO bugfix#1233

Open
Kenneth-T-Moore wants to merge 21 commits into
OpenMDAO:mainfrom
Kenneth-T-Moore:rho_sea_level
Open

Sea Level Density overhaul -> Requires OpenMDAO bugfix#1233
Kenneth-T-Moore wants to merge 21 commits into
OpenMDAO:mainfrom
Kenneth-T-Moore:rho_sea_level

Conversation

@Kenneth-T-Moore

Copy link
Copy Markdown
Member

Summary

Remove the RHO_SEA_LEVEL_ENGLISH and RHO_SEA_LEVEL_SI constants that were used in various places and had different values.

  • In places where this was used in the denominator of a density ratio, it was replaced with an option Mission.SEA_LEVEL_DENSITY. This option will be set automatically by the preprocessor if it isn't specified in the csv file. Like gravity, the preprocessor will compute it from the atmosphere model that is chosen.
  • In places where Density is meant to be the current state, it is now pulled from an existing atmosphere component instead.

In 2DOF, thee VRot execcomp, which computes the speed at which rotation starts, has been moved into the groundroll phase along with its constraint.

  • The execcomp has been replaced with an ExplicitComponent in preparation for the gravity overhaul part 4.
  • The density input now uses the local atmosphere instead of a hard-coded value.
  • The mass input now uses the mass at rotation instead of at taxi end.

This work has exposed a bug in OpenMDAO when connecting an input to another input. This requires a bugfix that is currently a PR, but will soon be merged, and will eventually become part of a release. The CI has been configured to load OpenMDAO from github in all workflows for now.

Related Issues

Backwards incompatibilities

None

AI Usage

None

Comment thread aviary/mission/two_dof/ode/takeoff_ode.py
@Kenneth-T-Moore Kenneth-T-Moore marked this pull request as ready for review July 16, 2026 17:22
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.

Remove RHO_SEA_LEVEL constant

2 participants