-
Notifications
You must be signed in to change notification settings - Fork 52
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[MAINT] Remove conda from CI Linux Workflows (#794)
* removed conda from ci pipeline * MAINT: Switching to ubuntu 22.04 * MAINT: Added conditionals to install openmpi according to matrix.os * MAINT: moving macos MPI_LIB_NRN_PATH env variable assigment * MAINT: Removing comments and additional test in unit_test.yml * MAINT: Moving linux CI to a separate workflow * STY: fixed flake8 errors * STY: fixed flake8 errors in gui.py * MAINT: Removing linux commands from macos CI workflow * TST: Adding python 3.10 and 3.11 to the tests * TST: Adding quotes to python version 3.10 * TST: Adding double quotes for macos python 3.10 build * MAINT: removing python 3.11 from CI tests * MAINT: Testing Ifs in workflow to test multiple OS * MAINT: Fixed checkout@v4 statement * MAINT: Adding python 3.9 and 3.10 to multi porpuse workflow * MAINT: Removed linux workflow file. Moved logic to unix_unit_tests.yml
- Loading branch information
Showing
4 changed files
with
71 additions
and
55 deletions.
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file contains 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 |
---|---|---|
@@ -0,0 +1,68 @@ | ||
name: Unit tests | ||
|
||
on: | ||
push: | ||
branches: ['**'] | ||
pull_request: | ||
branches: ['**'] | ||
|
||
jobs: | ||
build: | ||
|
||
runs-on: ${{ matrix.os }} | ||
strategy: | ||
matrix: | ||
os: [ubuntu-latest,macos-latest] | ||
python-version: [3.8, 3.9, '3.10'] | ||
|
||
steps: | ||
- uses: actions/checkout@v4 | ||
|
||
- name: Install Ubuntu System dependencies | ||
if: matrix.os == 'ubuntu-latest' | ||
shell: bash -el {0} | ||
run: | | ||
sudo add-apt-repository ppa:ubuntu-toolchain-r/test | ||
sudo apt-get update | ||
sudo apt-get install libopenmpi-dev openmpi-bin | ||
- name: Activate conda environment for macOS | ||
if: matrix.os == 'macos-latest' | ||
uses: conda-incubator/setup-miniconda@v3 | ||
with: | ||
activate-environment: test | ||
python-version: ${{ matrix.python-version }} | ||
fetch-depth: 2 | ||
|
||
- name: Activate Python environment for Ubuntu | ||
if: matrix.os == 'ubuntu-latest' | ||
uses: actions/setup-python@v5 | ||
with: | ||
python-version: ${{ matrix.python-version }} | ||
|
||
- name: Install Python dependencies | ||
shell: bash -el {0} | ||
run: | | ||
if [ ${{ matrix.os }} == 'macos-latest' ]; then | ||
python -m pip install --upgrade pip | ||
conda install --yes -c conda-forge mpi4py openmpi | ||
elif [[ "${{ matrix.os }}" == "ubuntu-latest" ]]; then | ||
python -m pip install --upgrade pip | ||
python -m pip install mpi4py | ||
fi | ||
- name: Install HNN-core | ||
shell: bash -el {0} | ||
run: | | ||
pip install --verbose '.[opt, parallel, test, gui]' | ||
- name: Lint with flake8 | ||
shell: bash -el {0} | ||
run: | | ||
flake8 --count hnn_core | ||
- name: Test with pytest | ||
shell: bash -el {0} | ||
run: | | ||
python -m pytest ./hnn_core/tests/ --cov=hnn_core --cov-report=xml | ||
- name: Upload coverage to Codecov | ||
shell: bash -el {0} | ||
run: | | ||
bash <(curl -s https://codecov.io/bash) -f ./coverage.xml |
This file contains 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 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