Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Aviary Version 0.9.6 #563

Draft
wants to merge 63 commits into
base: main
Choose a base branch
from
Draft

Aviary Version 0.9.6 #563

wants to merge 63 commits into from

Conversation

jkirk5
Copy link
Contributor

@jkirk5 jkirk5 commented Oct 9, 2024 •
edited
Loading

Summary

This PR collects non-backwards compatible changes incoming for v0.9.6

Backwards incompatibilities

  • Changed variable names in Dynamic category
  • Updated version requirements for OpenMDAO (>= 3.35.0), location of generated reports is now different and incompatible with runs generated using environments with older versions of OpenMDAO
  • Atmosphere now assumed geometric/geodetic altitude (previously defaulted to geopotential)

New Dependencies

None

jkirk5 and others added 21 commits September 5, 2024 15:17
propeller subclass in hierarchy
705b459
fixes for propeller variable name change
92ee764
dynamic var subclasses
31704f0
find/replace
07ea72f
find/replace fixes
fb49c99
additional find/replace fixes
2cdbf22 
sorted metadata
another round of find/replace fixes
65909d6
more test fixes
8675a85
doc fixes
a1fe0f1
Merge remote-tracking branch 'remotes/origin/main' into variable_names
1aaab01
Merge remote-tracking branch 'remotes/origin/main' into variable_names
64faaee
merge fixes
1ae3d4a
updated hierarchy categories
ecb7b10
metadata typo
b654403
@ehariton
Merge pull request #519 from jkirk5/variable_names
4d97781 
Variable hierarchy overhaul (propellers & Dynamic)
missed find+replace
6c13f59
Merge remote-tracking branch 'remotes/origin/main' into variable_names
a1f8ec2
more missing replacements
d20026e
another round post-merge updates
4d939ba
yet another missing replace
2339dac
@jkirk5
Merge pull request #561 from jkirk5/variable_names
2652e69 
Fixes for variable hierarchy update
@jkirk5 jkirk5 added the epic Long-term work that can't be resolved with single pull request label Oct 9, 2024
xjjiang and others added 30 commits October 12, 2024 14:07
@xjjiang
testing: roll back atmosphere.py
5ffae54
@xjjiang
It is confirmed that changing default of h_def will result in the fai…
c77a39b 
…lure of TurbopropTest.test_turboprop() of test_custom_engine_model.py.
@xjjiang
testing: try set default Dynamic.Mission.MASS = 100000
f6ea500
@xjjiang
roll back specific_energy_rate.py
9dc636a
@xjjiang
try to set Aircraft.Engine.GEOPOTENTIAL_ALT = False
7987e6e
@xjjiang
add a altitude bounds to test_custom_engine_model.py
5a7ae98
@xjjiang
set num_segments = 5 in test_custom_engine_model.py
7862d87
@xjjiang
try to set initial_altitude to 34000.0 ft in test_custom_engine_model.py
a646b7f
@xjjiang
remove altitude_bounds from test_custom_engine_model.py
5f639c6
@xjjiang
set back num_segments to 2 in test_custom_engine_model.py
4ef4b98
@xjjiang
set back num_segments to 3 in test_custom_engine_model.py
72a3f8e
@xjjiang
set back initial_altitude to 35000 in test_custom_engine_model.py
fce3cb1
@xjjiang
remove altitude_bounds in test_custom_engine_model.py
45ea155
@xjjiang
minor update
668ed55
@xjjiang
set max_iter = 150
251d546
@xjjiang
set max_iter = 100
c250ef1
@xjjiang
remove Aircraft.Engine.GEOPOTENTIAL_ALT which was added for testing.
0520aa8
@xjjiang
minor update
7c2dd1a
@xjjiang
Set desired values back to original per Ken's request. This regressio…
861cc0c 
…n is due to a different bug, related to the design range: issue #569. I have to adjust tolerances in order to pass unit tests.
Merge remote-tracking branch 'remotes/upstream/0.9.6.dev' into geodetic
e3283ea
merge fixes
a3844bb
@jkirk5
Merge pull request #567 from xjjiang/geodetic
dc2eead 
Change atmosphere comp defaults from geopotential to Geodetic
Merge remote-tracking branch 'remotes/origin/main' into 0.9.6.dev
5707397
Merge remote-tracking branch 'remotes/upstream/0.9.6.dev' into 0.9.6.dev
f317125
@jkirk5
Merge pull request #573 from jkirk5/0.9.6.dev
10b8cc9 
0.9.6.dev
merge fixes
83015c3
@jkirk5
Merge pull request #580 from jkirk5/0.9.6.dev
d1e6242 
merge fixes
alphabetization fix
33da3e3
Merge remote-tracking branch 'remotes/origin/main' into 0.9.6.dev
a23bc87
@jkirk5
Merge pull request #589 from jkirk5/0.9.6.dev
08f0c8c 
merge with main
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
epic Long-term work that can't be resolved with single pull request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jkirk5 @ehariton @xjjiang