Skip to content

Commit

Permalink
Merge pull request #642 from Kenneth-T-Moore/main
Browse files Browse the repository at this point in the history
Pin CI to Ubuntu 22 due to some incompatibility in the test workflows in the Ubuntu 24.04,
  • Loading branch information
jkirk5 authored Jan 8, 2025
2 parents 75c0804 + 670b50a commit 394fb52
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 21 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/test_benchmarks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ on:
jobs:

latest_benchmarks:
runs-on: ubuntu-latest
runs-on: ubuntu-22.04
timeout-minutes: 90

steps:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test_docs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ on:
jobs:

latest_docs:
runs-on: ubuntu-latest
runs-on: ubuntu-22.04
timeout-minutes: 90

steps:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/test_workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ jobs:

pre_commit:
# run pre-commit checks
runs-on: ubuntu-latest
runs-on: ubuntu-22.04

steps:
- uses: actions/checkout@v4
Expand All @@ -28,7 +28,7 @@ jobs:
- uses: pre-commit/action@v3.0.1

test_ubuntu:
runs-on: ubuntu-latest
runs-on: ubuntu-22.04
strategy:
fail-fast: false
matrix:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test_workflow_dev_deps.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ on:
jobs:

test_ubuntu:
runs-on: ubuntu-latest
runs-on: ubuntu-22.04
strategy:
fail-fast: false
matrix:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test_workflow_no_dev_install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ on:
jobs:

test_ubuntu_no_dev_install:
runs-on: ubuntu-latest
runs-on: ubuntu-22.04

timeout-minutes: 90

Expand Down
15 changes: 0 additions & 15 deletions aviary/utils/preprocessors.py
Original file line number Diff line number Diff line change
Expand Up @@ -400,21 +400,6 @@ def preprocess_propulsion(aviary_options: AviaryValues, engine_models: list = No
aviary_options.set_val(Aircraft.Engine.NUM_WING_ENGINES, num_wing_engines_all)
aviary_options.set_val(Aircraft.Engine.NUM_FUSELAGE_ENGINES, num_fuse_engines_all)

# Update nacelle-related variables in aero to be sized to the number of
# engine types.
if num_engine_type > 1:

keys = [
Aircraft.Nacelle.LAMINAR_FLOW_LOWER,
Aircraft.Nacelle.LAMINAR_FLOW_UPPER
]

for var in keys:
try:
aviary_options.get_val(var)
except KeyError:
aviary_options.set_val(var, np.zeros(num_engine_type))

if Mission.Summary.FUEL_FLOW_SCALER not in aviary_options:
aviary_options.set_val(Mission.Summary.FUEL_FLOW_SCALER, 1.0)

Expand Down

0 comments on commit 394fb52

Please sign in to comment.