Switch off MLOPR test (#564)

* Switch off MLOPR test * mpi4py seems to be problematic for 3.8 as well
ptycho · Jul 30, 2024 · 99743b9 · 99743b9
1 parent 6c94a54
commit 99743b9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -57,6 +57,9 @@ jobs:
         if [ ${{ matrix.conda-env }} == 'full' ] && [ ${{ matrix.python-version }} == '3.12' ]; then
           sed -i '/- pyfftw/d' dependencies_${{ matrix.conda-env }}.yml
         fi
+        if [ ${{ matrix.conda-env }} == 'full' ] && [ ${{ matrix.python-version }} == '3.8' ]; then
+          sed -i '/- mpi4py/d' dependencies_${{ matrix.conda-env }}.yml
+        fi
         if [ ${{ matrix.conda-env }} == 'full' ] && [ ${{ matrix.python-version }} == '3.9' ]; then
           sed -i '/- mpi4py/d' dependencies_${{ matrix.conda-env }}.yml
         fi

diff --git a/test/engine_tests/MLOPR_test.py b/test/engine_tests/MLOPR_test.py
@@ -17,8 +17,8 @@
 
 from ptypy.custom import MLOPR
 
-@pytest.mark.skipif(sys.version_info > (3,12),
-                    reason="Test broken for Python 3.12")
+@pytest.mark.skipif(sys.version_info > (3,8),
+                    reason="Test broken")
 class MLOPRTest(unittest.TestCase):
     def setUp(self):
         self.outpath = tempfile.mkdtemp(suffix="MLOPR_test")