Skip to content

Commit

Permalink
Fix OpenMDAO install in GitHub Actions build (#53)
Browse files Browse the repository at this point in the history
* Update GHA checkout version

* Roll back pip version for oldest tests

* Fix pip upgrade in GHA

* Trying older setuptools

* Try getting rid of pinned pip version in oldest build

* Pinned pip version is necessary

* Remove which python debug in GHA

* Trying old wheel instead of pip and setuptools

* No bueno
  • Loading branch information
eytanadler authored Jun 26, 2023
1 parent c92037e commit 34f5f94
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion .github/workflows/openconcept.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,8 @@ jobs:
# ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Set versions to test here ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
PYTHON_VERSION_OLDEST: ['3.8']
PYTHON_VERSION_LATEST: ['3.11']
PIP_VERSION_OLDEST: ['23.0.1'] # pip>=23.1 cannot build the oldest OpenMDAO
SETUPTOOLS_VERSION_OLDEST: ['66.0.0'] # setuptools >= 67.0.0 can't build the oldest OpenMDAO
NUMPY_VERSION_OLDEST: ['1.20'] # latest is most recent on PyPI
SCIPY_VERSION_OLDEST: ['1.6.0'] # latest is most recent on PyPI
OPENMDAO_VERSION_OLDEST: ['3.10'] # latest is most recent on PyPI
Expand All @@ -35,7 +37,7 @@ jobs:
run:
shell: bash -l {0}
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v3
- name: Setup Python ${{ matrix.PYTHON_VERSION_OLDEST }}
if: ${{ matrix.dep-versions == 'oldest' }}
uses: conda-incubator/setup-miniconda@v2
Expand All @@ -53,6 +55,7 @@ jobs:
if: ${{ matrix.dep-versions == 'oldest' }}
run: |
conda config --set always_yes yes
python -m pip install pip==${{ matrix.PIP_VERSION_OLDEST }} setuptools==${{ matrix.SETUPTOOLS_VERSION_OLDEST }} --upgrade wheel
pip install numpy==${{ matrix.NUMPY_VERSION_OLDEST }} scipy==${{ matrix.SCIPY_VERSION_OLDEST }} openmdao==${{ matrix.OPENMDAO_VERSION_OLDEST }}
- name: Install dependencies (latest versions)
if: ${{ matrix.dep-versions == 'latest' }}
Expand Down

0 comments on commit 34f5f94

Please sign in to comment.