Skip to content

Commit f092591

Browse files
authored
Fix: Workflow maintaining conda environment (NOAA-GFDL#146)
* Updating test workflow to use conda environment for addition of pyrte-rrtmgp to pySHiELD * Removing external install of mpich * Trying conda install of mpich to CI * Trying conda install of mpi4py to CI * Updating NDSL * Checking effect of updating NDSL on workflow * For testing * Undoing changes for testing pyshield * More testing in pyshield * Pace tests pass in pyshield call * Adding oversubscribe to main_unit_tests * Keeping sqlite in workflow * Removing oversubscribe
1 parent a316389 commit f092591

File tree

1 file changed

+4
-9
lines changed

1 file changed

+4
-9
lines changed

.github/workflows/main_unit_tests.yaml

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,9 @@ concurrency:
2626
jobs:
2727
pace_main_unit_tests:
2828
runs-on: ubuntu-latest
29+
defaults:
30+
run:
31+
shell: bash -el {0}
2932
steps:
3033
- name: "External trigger: Checkout pace/develop"
3134
if: ${{ inputs.component_trigger }}
@@ -45,11 +48,9 @@ jobs:
4548
activate-environment: pace_test_env
4649

4750
- name: Install mpi (MPICH flavor)
48-
shell: bash -l {0}
4951
run: pip install mpich
5052

5153
- name: "External trigger: Remove existing component in pace/develop"
52-
shell: bash -l {0}
5354
if: ${{ inputs.component_trigger }}
5455
run: rm -rf ${GITHUB_WORKSPACE}/pace/${{inputs.component_name}}
5556

@@ -60,35 +61,29 @@ jobs:
6061
path: pace/${{inputs.component_name}}
6162

6263
- name: Install packages
63-
shell: bash -l {0}
6464
run: |
6565
cd ${GITHUB_WORKSPACE}/pace
6666
pip install .[test]
6767
pip install numpy==1.26.4
68-
conda remove sqlite
6968
7069
- name: Print versions
71-
shell: bash -l {0}
7270
run: |
7371
python --version
7472
pip --version
7573
pip list
7674
7775
- name: Prepare input files
78-
shell: bash -l {0}
7976
run: |
8077
cd ${GITHUB_WORKSPACE}/pace
8178
mkdir tests/main/input
8279
python examples/generate_eta_files.py tests/main/input
8380
8481
- name: Run tests
85-
shell: bash -l {0}
8682
run: |
8783
cd ${GITHUB_WORKSPACE}/pace
8884
python -m pytest -x tests/main
8985
9086
- name: Run baroclinic test case
91-
shell: bash -l {0}
9287
run: |
9388
cd ${GITHUB_WORKSPACE}/pace
94-
mpiexec -np 6 --oversubscribe python -m pace.run examples/configs/baroclinic_c48_no_out.yaml
89+
mpiexec -np 6 python -m pace.run examples/configs/baroclinic_c48_no_out.yaml

0 commit comments

Comments
 (0)