Merge pull request #47 from HideakiImamura/tests/remove-stale-integra…

…tion-tests Remove tests for allennlp and chainer
optuna · Nov 17, 2023 · e8e3718 · e8e3718
2 parents bf16b34 + f87c5de
commit e8e3718
Show file tree

Hide file tree

Showing 21 changed files with 2 additions and 1,490 deletions.
diff --git a/.github/workflows/checks.yml b/.github/workflows/checks.yml
@@ -27,7 +27,6 @@ jobs:
     - name: Setup environment
       run: |
         sudo apt-get update
-        sudo apt-get -y install openmpi-bin libopenmpi-dev
 
     - name: Install
       run: |

diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml
@@ -40,7 +40,7 @@ jobs:
     - name: Setup environment
       run: |
         sudo apt-get update
-        sudo apt-get -y install openmpi-bin libopenmpi-dev libopenblas-dev
+        sudo apt-get -y install libopenblas-dev
 
     - name: Install
       run: |
@@ -71,23 +71,10 @@ jobs:
         COVERAGE_COVERAGE: yes  # https://github.com/nedbat/coveragepy/blob/65bf33fc03209ffb01bbbc0d900017614645ee7a/coverage/control.py#L255-L261
       run: |
         coverage run --source=optuna -m pytest tests \
-          --ignore tests/allennlp_tests \
           --ignore tests/test_mxnet.py
         coverage combine
         coverage xml
 
-    - name: Multi-node tests
-      env:
-        OMP_NUM_THREADS: 1
-        PYTHONPATH: .
-        COVERAGE_PROCESS_START: .coveragerc
-        COVERAGE_COVERAGE: yes
-      run: |
-        export OMPI_MCA_rmaps_base_oversubscribe=yes
-        mpirun -n 2 -- coverage run -m pytest tests/test_chainermn.py
-        coverage combine --append
-        coverage xml
-
     - name: Upload coverage to Codecov
       uses: codecov/codecov-action@v3
       with:

diff --git a/.github/workflows/mac-tests.yml b/.github/workflows/mac-tests.yml
@@ -54,8 +54,6 @@ jobs:
 
     - name: Setup mac environment
       run: |
-        brew install libomp
-        brew install open-mpi
         brew install openblas
 
     - name: Install
@@ -86,20 +84,11 @@ jobs:
     - name: Tests
       run: |
         pytest tests \
-        --ignore tests/allennlp_tests \
         --ignore tests/test_mxnet.py
       env:
         OMP_NUM_THREADS: 1
 
-    - name: Tests MPI
-      run: |
-        export OMPI_MCA_rmaps_base_oversubscribe=yes
-        mpirun -n 2 -- pytest tests/test_chainermn.py
-      env:
-        OMP_NUM_THREADS: 1
-
     - name: Tests(Deprecated)
       run: |
         pip install "numpy<1.24"
-        pytest tests/allennlp_tests
         pytest tests/test_mxnet.py
diff --git a/.github/workflows/tests-mpi.yml b/.github/workflows/tests-mpi.yml
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -53,7 +53,7 @@ jobs:
     - name: Setup environment
       run: |
         sudo apt-get update
-        sudo apt-get -y install openmpi-bin libopenmpi-dev libopenblas-dev
+        sudo apt-get -y install libopenblas-dev
 
     - name: Install
       run: |
@@ -87,13 +87,9 @@ jobs:
     - name: Tests
       run: |
         pytest tests \
-          --ignore tests/allennlp_tests \
           --ignore tests/test_mxnet.py
 
     - name: Tests(Deprecated)
       run: |
         pip install "numpy<1.24"
-        if [ ${{ matrix.python-version }} != 3.11 ]; then
-          pytest tests/allennlp_tests
-        fi
         pytest tests/test_mxnet.py
diff --git a/pyproject.toml b/pyproject.toml
@@ -52,11 +52,8 @@ document = [
     "sphinx_rtd_theme",
 ]
 all = [
-  "allennlp>=2.2.0; python_version<'3.11'",
   "catalyst",
-  "chainer>=5.0.0",
   "fastai",
-  "mpi4py",
   "mxnet",
   "shap",
   "skorch",

diff --git a/tests/allennlp_tests/__init__.py b/tests/allennlp_tests/__init__.py
diff --git a/tests/allennlp_tests/example.jsonnet b/tests/allennlp_tests/example.jsonnet
diff --git a/tests/allennlp_tests/example_with_environment_variables.jsonnet b/tests/allennlp_tests/example_with_environment_variables.jsonnet
diff --git a/tests/allennlp_tests/example_with_executor_and_pruner.jsonnet b/tests/allennlp_tests/example_with_executor_and_pruner.jsonnet