Merge pull request #981 from fabratu/20220819_fix_gha_notebooks

GHA: Fix errors for notebook tests and activate error output on std.
networkit · Aug 22, 2022 · 6d970b9 · 6d970b9
2 parents 1e4b616 + 91b120e
commit 6d970b9
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -387,7 +387,7 @@ jobs:
           cd ../..
           python -m venv pyenv
           . ./pyenv/Scripts/activate
-          pip install cython ipython jupyter
+          pip install cython ipython jupyter ipywidgets==7.7.1 jupyterlab-widgets==1.1.1
           mkdir build && cd build
           cmake -GNinja -DNETWORKIT_BUILD_TESTS=ON -DNETWORKIT_STATIC=ON -DCMAKE_BUILD_TYPE=Release -DNETWORKIT_CXX_STANDARD=${{ env.CXX_STANDARD }} -DNETWORKIT_WARNINGS=ON -DNETWORKIT_WARNINGS_AS_ERRORS=ON -DNETWORKIT_EXT_TLX=${{ env.TLX_PATH_WIN }} -DNETWORKIT_NATIVE=${{ env.NATIVE }} ..
           ninja

diff --git a/.github/workflows/scripts/full.sh b/.github/workflows/scripts/full.sh
@@ -40,6 +40,8 @@ pip3 install -r requirements.txt
 
 python3 -m unittest discover -v networkit/test/
 
-pip3 install ipycytoscape plotly seaborn
+# Installing ipywidgets and jupyterlab-widgets (+ pinned versions) is temp. necessary due to incompatibility with Jupyterlab/plotly and ipywidgets>8.X
+# See: https://github.com/plotly/plotly.py/issues/3686
+pip3 install ipycytoscape plotly seaborn ipywidgets==7.7.1 jupyterlab-widgets==1.1.1
 
 python3 notebooks/test_notebooks.py 'notebooks/'
diff --git a/notebooks/test_notebooks.py b/notebooks/test_notebooks.py
@@ -36,6 +36,7 @@ def run_notebook(path):
         return 0
     else:
         print(" " + path + " test exited with errors.")
+        print(errors)
         return 1
 
 if __name__ == '__main__':