Skip to content

Remove RHO_SEA_LEVEL constant #1191

Description

@jkirk5

Desired capability or behavior.

The constant RHO_SEA_LEVEL appears to be unnecessary and can be removed.

Suggested implementation.

Currently the constant is being used in a couple different ways. First is as a hardcoded air density for some components that assume sea level altitude such as energy takeoff/landing - these components should be updated to use the full atmosphere component and accept any altitude. Second is in several places throughout the code to compute a density ratio of actual density / sea level density. This also shouldn't be hardcoded (since sea-level density changes depending on which atmosphere model is being used). I don't have a implementation idea for replacing the use of the constant in this area yet.

Metadata

Metadata

Labels

code cleanupCode cleanup, refactoring, or similar helpful reorganizationrequestRequest for new feature or change

Type

No type

Fields

No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions