Fix CI

Signed-off-by: Igoshev, Iaroslav <iaroslav.igoshev@intel.com>
YarShev · Nov 13, 2023 · 092b925 · 092b925
1 parent 89c0473
commit 092b925
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 6 deletions.
diff --git a/.github/workflows/ci-notebooks.yml b/.github/workflows/ci-notebooks.yml
@@ -29,7 +29,7 @@ jobs:
     steps:
       - uses: actions/checkout@v3
       - uses: ./.github/actions/python-only
-        if: matrix.execution != 'hdk_on_native' || matrix.execution != 'pandas_on_unidist'
+        if: matrix.execution != 'hdk_on_native' && matrix.execution != 'pandas_on_unidist'
       - uses: ./.github/actions/mamba-env
         with:
           environment-file: requirements/env_hdk.yml
@@ -49,25 +49,25 @@ jobs:
       # replace modin with . in the tutorial requirements file for `pandas_on_ray` and
       # `pandas_on_dask` since we need Modin built from sources
       - run: sed -i 's/modin/./g' examples/tutorial/jupyter/execution/${{ matrix.execution }}/requirements.txt
-        if: matrix.execution != 'hdk_on_native' || matrix.execution != 'pandas_on_unidist'
+        if: matrix.execution != 'hdk_on_native' && matrix.execution != 'pandas_on_unidist'
       # install dependencies required for notebooks execution for `pandas_on_ray` and `pandas_on_dask`
       # Override modin-spreadsheet install for now
       - run: |
           pip install -r examples/tutorial/jupyter/execution/${{ matrix.execution }}/requirements.txt
           pip install git+https://github.com/modin-project/modin-spreadsheet.git@49ffd89f683f54c311867d602c55443fb11bf2a5
-        if: matrix.execution != 'hdk_on_native' || matrix.execution != 'pandas_on_unidist'
+        if: matrix.execution != 'hdk_on_native' && matrix.execution != 'pandas_on_unidist'
       # Build Modin from sources for `hdk_on_native` and `pandas_on_unidist`
       - run: pip install -e .
         if: matrix.execution == 'hdk_on_native' || matrix.execution == 'pandas_on_unidist'
       # install test dependencies
       # NOTE: If you are changing the set of packages installed here, make sure that
       # the dev requirements match them.
       - run: pip install pytest pytest-cov black flake8 flake8-print flake8-no-implicit-concat
-        if: matrix.execution != 'hdk_on_native' || matrix.execution != 'pandas_on_unidist'
+        if: matrix.execution != 'hdk_on_native' && matrix.execution != 'pandas_on_unidist'
       - run: pip install flake8-print jupyter nbformat nbconvert
         if: matrix.execution == 'hdk_on_native' || matrix.execution == 'pandas_on_unidist'
       - run: pip list
-        if: matrix.execution != 'hdk_on_native' || matrix.execution != 'pandas_on_unidist'
+        if: matrix.execution != 'hdk_on_native' && matrix.execution != 'pandas_on_unidist'
       - run: |
           conda info
           conda list

diff --git a/.github/workflows/ci-required.yml b/.github/workflows/ci-required.yml
@@ -32,6 +32,11 @@ jobs:
           architecture: "x64"
           cache: "pip"
           cache-dependency-path: '**/requirements-doc.txt'
+      # Install a working MPI implementation so mpi4py can link to it
+      - name: Install MPICH
+        run: |
+          sudo apt install libmpich-dev
+          which mpiexec
       - run: pip install -r docs/requirements-doc.txt
       - run: cd docs && sphinx-build -T -E -W -b html . build
 

diff --git a/docs/requirements-doc.txt b/docs/requirements-doc.txt
@@ -1,5 +1,5 @@
 # install current modin checkout to bring all required dependencies
-.[all]
+.[all,mpi]
 # now install some more optional dependencies
 colorama
 click