Merge pull request #102 from Pressio/develop

merge Develop into main

Author: eparish1

.github/workflows/install-and-test.yaml

@@ -7,8 +7,8 @@ on:
     branches: [main, develop]
 
 concurrency:
-  group: CI-${{ github.head_ref }}
-  cancel-in-progress: true
+  group: ${{ github.event.repository.name }}-${{ github.ref }}-${{ github.workflow }}
+  cancel-in-progress: True
 
 jobs:
   test:
@@ -17,15 +17,42 @@ jobs:
       fail-fast: false # true -> cancel all jobs if any fails
       matrix:
         python-version: ["3.8", "3.9", "3.10", "3.11"]
+        mpi: ['mpich','openmpi']
+        exclude:
+          - python-version: "3.8"
+            mpi: 'mpich'
+          - python-version: "3.9"
+            mpi: 'mpich'
 
     steps:
-      - uses: actions/checkout@v3
+      - name: Check out code
+        uses: actions/checkout@v3
+        with:
+          fetch-depth: 0
+
+      - name: Check for MPI tests with invalid rank count
+        run: |
+          python $GITHUB_WORKSPACE/helper_scripts/check-mpi-size.py
 
       - name: Set up Python ${{ matrix.python-version }}
         uses: actions/setup-python@v4
         with:
           python-version: ${{ matrix.python-version }}
 
+      - name: Setup MPI
+        uses: mpi4py/setup-mpi@v1
+        with:
+          mpi: ${{ matrix.mpi }}
+
+      - name: Install mpi4py
+        run: |
+          python -m pip install mpi4py
+
+      - name: Set Environment Variables
+        run: |
+          echo "OMPI_ALLOW_RUN_AS_ROOT=1" >> $GITHUB_ENV
+          echo "OMPI_ALLOW_RUN_AS_ROOT_CONFIRM=1" >> $GITHUB_ENV
+
       - name: Check environment
         run: |
           echo ======================================================
@@ -49,6 +76,24 @@ jobs:
           pip list
           pip install .
 
-      - name: Run tests
+      - name: Check Formatting (Pylint)
+        run: |
+          pip install pylint
+          pylint --recursive y --rcfile .pylintrc --output-format=colorized ./romtools
+        continue-on-error: false
+
+      - name: Run tests (Serial)
+        run: |
+          pytest --timeout=300
+
+      - name: Run tests (MPI, n=1)
+        run: |
+          mpirun -n 1 python -m pytest tests/romtools/* --with-mpi --timeout=300
+
+      - name: Run tests (MPI, n=3)
+        run: |
+          mpirun -n 3 python -m pytest tests/romtools/* --with-mpi --timeout=300
+
+      - name: Run tests (MPI, n=4)
         run: |
-          pytest -s
+          mpirun -n 4 python -m pytest tests/romtools/* --with-mpi --timeout=300