Skip to content

Mass report update - FLOPS#942

Merged
jkirk5 merged 42 commits intoOpenMDAO:mainfrom
jkirk5:reports
Feb 4, 2026
Merged

Mass report update - FLOPS#942
jkirk5 merged 42 commits intoOpenMDAO:mainfrom
jkirk5:reports

Conversation

@jkirk5
Copy link
Copy Markdown
Contributor

@jkirk5 jkirk5 commented Dec 19, 2025

Summary

Updates mass reporting for FLOPS-based mass to conform to SAWE RP-8 standard.
Some mass variables have been renamed to comply with the standard, and the FLOPS-based mass summation component was reworked to group masses appropriately.

Generative AI was used to write a new test that checks matching between hierarchy variable names and string representation (test_var_structure.py). The Cursor IDE was used, which does not expose which specific models it uses. Manual review and minor tweaks were done by hand.

Related Issues

Backwards incompatibilities

Renamed variables:
Aircraft.CrewPayload.NUM_TOURIST_CLASS -> Aircraft.CrewPayload.NUM_ECONOMY_CLASS
Aircraft.CrewPayload.NON_FLIGHT_CREW_MASS -> Aircraft.CrewPayload.CABIN_CREW_MASS
Aircraft.CrewPayload.NON_FLIGHT_CREW_MASS_SCALER -> Aircraft.CrewPayload.CABIN_CREW_MASS_SCALER
Aircraft.CrewPayload.PASSENGER_MASS -> Aircraft.CrewPayload.PASSENGER_MASS_TOTAL
Aircraft.CrewPayload.PASSENGER_MASS_WITH_BAGS -> Aircraft.CrewPayload.MASS_PER_PASSENGER_WITH_BAGS
Aircraft.Controls.TOTAL_MASS -> Aircraft.Controls.MASS
Aircraft.Design.SYSTEMS_EQUIP_MASS -> Aircraft.Design.SYSTEMS_AND_EQUIPMENT_MASS
Aircraft.Design.SYSTEMS_EQUIP_MASS_BASE -> Aircraft.Design.SYSTEMS_AND_EQUIPMENT_MASS_BASE
Aircraft.Wing.ASPECT_RATIO_REF -> Aircraft.Wing.ASPECT_RATIO_REFERENCE
Aircraft.Wing.THICKNESS_TO_CHORD_REF -> Aircraft.Wing.THICKNESS_TO_CHORD_REFERENCE

New Dependencies

None

Comment thread aviary/subsystems/mass/flops_based/landing_gear.py Outdated
Comment thread aviary/subsystems/mass/flops_based/test/test_mass_summation.py Outdated
Copy link
Copy Markdown
Member

@Kenneth-T-Moore Kenneth-T-Moore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Surprised the variable match test didn't find more mistakes in the names. Guess they are all good now.

Note: there is a test regression that should be investigated.

@jkirk5 jkirk5 enabled auto-merge January 21, 2026 15:25
@jkirk5 jkirk5 added this pull request to the merge queue Feb 4, 2026
Merged via the queue into OpenMDAO:main with commit 0db679b Feb 4, 2026
6 checks passed
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.

2 participants