-
Notifications
You must be signed in to change notification settings - Fork 121
Turboprop freighter fixes #1098
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
Merged
Merged
Changes from all commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
da4eeaf
fixes for electric test case
jkirk5 978ad0d
removed hardcoded design vars from motor model
jkirk5 34c4f0e
minor updates to electrified turboprop test case
jkirk5 d678d3c
Merge latest, resolve conflicts
Kenneth-T-Moore f1d0f48
Merge branch 'ham' of github.com:Kenneth-T-Moore/aviary into jason_turbo
Kenneth-T-Moore 50fa7b7
jason_t
Kenneth-T-Moore 13bdce1
cleanup
Kenneth-T-Moore 63a8d36
debugging
Kenneth-T-Moore 03e1d67
debugging
Kenneth-T-Moore 9d41d0e
Merge branch 'ham' of github.com:Kenneth-T-Moore/aviary into jason_turbo
Kenneth-T-Moore f34a0b8
Merge branch 'ham' of github.com:Kenneth-T-Moore/aviary into jason_turbo
Kenneth-T-Moore 850d286
Merge branch 'ham' of github.com:Kenneth-T-Moore/aviary into jason_turbo
Kenneth-T-Moore ee82900
Merge branch 'ham' of github.com:Kenneth-T-Moore/aviary into jason_turbo
Kenneth-T-Moore 602e985
Debugging
Kenneth-T-Moore 68ede9f
Merge latest
Kenneth-T-Moore 492d58d
Merge branch 'main' of github.com:OpenMDAO/Aviary into jason_turbo
Kenneth-T-Moore 2d24043
Lower drag for the turboprop problem
Kenneth-T-Moore 4a8ed06
Merge branch 'main' of github.com:OpenMDAO/Aviary into jason_turbo
Kenneth-T-Moore dd33703
merge in real fix for the broken constraint in 2dof
Kenneth-T-Moore 99d3b8c
Merge pull request #29 from Kenneth-T-Moore/jason_turbo
jkirk5 dcd40ab
Merge branch 'main' into turboprop_freighter_fixes
jkirk5 2b1c649
Merge branch 'main' into turboprop_freighter_fixes
jkirk5 e2dbb96
updates for latest merge
jkirk5 8d3b18b
moved motor data file to correct location
jkirk5 d8b557c
A few small fixes.
Kenneth-T-Moore a9ab293
Added pack initial guess.
Kenneth-T-Moore 2b31ccf
Merge pull request #30 from Kenneth-T-Moore/jason_turbo2
jkirk5 87f3c42
typehints for subsystembuilder
jkirk5 9e8d05f
builder updates
jkirk5 90d18e0
returned kwargs to reports
jkirk5 d63a595
rpm control electroprop wip
jkirk5 af211ef
debugging motor RPM control
jkirk5 e3bd336
Fixed a couple of bugs
Kenneth-T-Moore e09ce7a
Merge pull request #31 from Kenneth-T-Moore/jason_turbo3
jkirk5 a86057e
Fix an indentation error in the Hamilton Standard interps that affect…
Kenneth-T-Moore b2653a0
Merge pull request #32 from Kenneth-T-Moore/jason_turbo3
jkirk5 da484f5
propeller RPM connection bug fixed
jkirk5 85d054d
Merge branch 'turboprop_freighter_fixes' of https://github.com/jkirk5…
jkirk5 c1ce2bc
RPM & throttle pairing in motor model
jkirk5 4f8bd0c
Merge branch 'main' into turboprop_freighter_fixes
jkirk5 2f25d36
removed redundant max calc from motor
jkirk5 570bdc2
remove obsolete kwargs references in turboprop model
jkirk5 99f52ba
fix turboprop test
jkirk5 cb6cbc8
experimental motor fixes
jkirk5 9b88581
electric turboprop model updates
jkirk5 985e19a
fixed mistake in phase info user options
jkirk5 50f4cf6
update subsystem name typo
jkirk5 a3f4a01
hybrid turboprop first cut
jkirk5 b018516
CI test fixes
jkirk5 0cbb5a7
Merge branch 'main' into turboprop_freighter_fixes
jkirk5 93ff858
adjusted value for breguet cruise test that seemed to be right on the…
jkirk5 b0308c0
updated test values. Claude Opus refactor of individual asserts into …
jkirk5 1fc0488
Merge branch 'turboprop_freighter_fixes' of https://github.com/jkirk5…
jkirk5 f31cb3b
remove cell outputs
jkirk5 ef6bd08
remove unnecessary diag_partials
jkirk5 89be402
revert unnecessary change
jkirk5 b5f1141
Merge branch 'main' into turboprop_freighter_fixes
jkirk5 1b6f5fd
Merge branch 'main' into turboprop_freighter_fixes
jkirk5 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -143,8 +143,6 @@ aviary/reports/ | |
|
|
||
| #VSCode user settings | ||
| .vscode/ | ||
| #VSCode extensions | ||
| .continue/ | ||
|
|
||
| # Built docs | ||
| _build/ | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,13 +1,10 @@ | ||
| from aviary.variable_info.enums import SpeedType, PhaseType, ThrottleAllocation | ||
| from aviary.variable_info.enums import PhaseType, SpeedType | ||
|
|
||
| # Energy method | ||
| energy_phase_info = { | ||
| 'pre_mission': { | ||
| 'include_takeoff': False, | ||
| 'optimize_mass': True, | ||
| }, | ||
| 'pre_mission': {'include_takeoff': False, 'optimize_mass': True}, | ||
| 'climb': { | ||
| 'subsystem_options': {'core_aerodynamics': {'method': 'cruise', 'solve_alpha': 'true'}}, | ||
| 'subsystem_options': {'aerodynamics': {'method': 'cruise', 'solve_alpha': True}}, | ||
| 'user_options': { | ||
| 'num_segments': 5, | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If this system is having trouble converging, I'd say reduce the number of segments here. |
||
| 'order': 3, | ||
|
|
@@ -20,16 +17,18 @@ | |
| 'no_descent': True, | ||
| 'mass_ref': (154000, 'lbm'), | ||
| 'throttle_enforcement': 'control', | ||
| 'throttle_bounds': ((0.2, 1), 'unitless'), | ||
| 'time_initial_bounds': ((0.0, 0.0), 'min'), | ||
| 'time_duration_bounds': ((24.0, 128.0), 'min'), | ||
| }, | ||
| 'initial_guesses': { | ||
| 'altitude': ([100.0, 21_000.0], 'ft'), | ||
| 'mach': ([0.17, 0.475], 'unitless'), | ||
| 'throttle': ([1, 1], 'unitless'), | ||
| }, | ||
| }, | ||
| 'cruise': { | ||
| 'subsystem_options': {'core_aerodynamics': {'method': 'cruise', 'solve_alpha': 'true'}}, | ||
| 'subsystem_options': {'aerodynamics': {'method': 'cruise', 'solve_alpha': True}}, | ||
| 'user_options': { | ||
| 'num_segments': 5, | ||
| 'order': 3, | ||
|
|
@@ -41,16 +40,18 @@ | |
| 'altitude_bounds': ((20_000.0, 22_000.0), 'ft'), | ||
| 'mass_ref': (154000, 'lbm'), | ||
| 'throttle_enforcement': 'control', | ||
| 'throttle_bounds': ((0.2, 1), 'unitless'), | ||
| 'time_initial_bounds': ((24.0, 128.0), 'min'), | ||
| 'time_duration_bounds': ((56.5, 1000.0), 'min'), | ||
| }, | ||
| 'initial_guesses': { | ||
| 'altitude': ([21_000, 21_000.0], 'ft'), | ||
| 'mach': ([0.475, 0.475], 'unitless'), | ||
| 'throttle': ([1, 1], 'unitless'), | ||
| }, | ||
| }, | ||
| 'descent': { | ||
| 'subsystem_options': {'core_aerodynamics': {'method': 'cruise', 'solve_alpha': 'true'}}, | ||
| 'subsystem_options': {'aerodynamics': {'method': 'cruise', 'solve_alpha': True}}, | ||
| 'user_options': { | ||
| 'num_segments': 5, | ||
| 'order': 3, | ||
|
|
@@ -65,9 +66,11 @@ | |
| 'mass_ref': (154000, 'lbm'), | ||
| 'no_climb': True, | ||
| 'throttle_enforcement': 'control', | ||
| 'throttle_bounds': ((0.15, 1), 'unitless'), | ||
| 'time_initial_bounds': ((80, 1056.5), 'min'), | ||
| 'time_duration_bounds': ((29.0, 128.0), 'min'), | ||
| }, | ||
| 'initial_guesses': {'throttle': ([0.5, 0.15], 'unitless')}, | ||
| }, | ||
| 'post_mission': { | ||
| 'include_landing': False, | ||
|
|
@@ -253,19 +256,19 @@ | |
| 'throttle': ([0.956, 0.956], 'unitless'), | ||
| }, | ||
| }, | ||
| 'electric_cruise': { | ||
| 'cruise': { | ||
| 'user_options': { | ||
| 'phase_type': PhaseType.BREGUET_RANGE, | ||
| 'phase_type': PhaseType.SIMPLE_CRUISE, | ||
| 'alt_cruise': (21_000, 'ft'), | ||
| 'mach_cruise': 0.475, | ||
| 'mass_bounds': ((0, None), 'lbm'), | ||
| 'mass_ref': (150_000, 'lbm'), | ||
| 'time_duration_bounds': ((0.0, 15.0), 'h'), | ||
| 'time_duration_ref': (8, 'h'), | ||
| }, | ||
| 'initial_guesses': { | ||
| # [Initial mass, delta mass] for special cruise phase. | ||
| 'mass': ([150_000.0, -35_000], 'lbm'), | ||
| 'initial_distance': (100.0e3, 'ft'), | ||
| 'initial_time': (1_000.0, 's'), | ||
| 'altitude': (21_000, 'ft'), | ||
| 'mach': (0.475, 'unitless'), | ||
| 'mass': ([150_000.0, 115000], 'lbm'), | ||
| 'time': ([1516.0, 2100.0], 's'), | ||
| }, | ||
| }, | ||
| 'desc1': { | ||
|
|
@@ -277,7 +280,7 @@ | |
| 'input_speed_type': SpeedType.MACH, | ||
| 'time_duration_bounds': ((300.0, 1800.0), 's'), | ||
| 'time_duration_ref': (1000, 's'), | ||
| 'altitude_initial': (21_000, 'ft'), | ||
| # 'altitude_initial': (21_000, 'ft'), | ||
| 'altitude_final': (10_000, 'ft'), | ||
| 'altitude_bounds': ((10000.0, 21_000.0), 'ft'), | ||
| 'altitude_ref': (20_000, 'ft'), | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you sure you want to take these constraints out? I thought we also had an option in the ODEs to turn this constraint on vs. off.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ken removed them to help with convergence. Throttle should already have a path constraint as far as I'm aware, so I think this was overconstraining the problem