Change usage of 'rho' to Dynamic.Mission.DENSITY

Open johnjasa opened this issue 1 year ago • 2 comments

Description

In at least takeoff_ode.py we use 'rho' when we should be using Dynamic.Atmosphere.DENSITY. This led to confusion for users who were trying to add timeseries outputs for the atmospheric model but couldn't access Dynamic.Atmosphere.DENSITY.

Also other spots for similar issues, at least for 'rho'.

added by jkirk5 There are several places beyond takeoff_ode where rho is being used as an openmdao variable. Use of rho is fine as a python variable (but density is preferred), however any inputs/outputs to components must be updated to use the proper variable name

Example

See above

Aviary Version

0.9.4-dev

Relevant environment information

No response

Mar 25 '24 19:03 johnjasa

Is this issue solved?

Aug 19 '24 23:08 xjjiang

Is this issue solved?

No, rho is still being used in several places in the code

Nov 25 '24 15:11 jkirk5