Skip to content

Commit

Permalink
Removes lines that skip and run specific tests individually to work a…
Browse files Browse the repository at this point in the history
…round OOM bug in CI, needed to merge a prior PR unrelated to the OOM bug.
  • Loading branch information
rlratzel committed Dec 21, 2023
1 parent 8d5bba3 commit a5f4a72
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 16 deletions.
5 changes: 0 additions & 5 deletions ci/test_python.sh
Original file line number Diff line number Diff line change
Expand Up @@ -111,11 +111,6 @@ popd
rapids-logger "pytest networkx using nx-cugraph backend"
pushd python/nx-cugraph
./run_nx_tests.sh
# Individually run tests that are skipped above b/c they may run out of memory
PYTEST_NO_SKIP=True ./run_nx_tests.sh --cov-append -k "TestDAG and test_antichains"
PYTEST_NO_SKIP=True ./run_nx_tests.sh --cov-append -k "TestMultiDiGraph_DAGLCA and test_all_pairs_lca_pairs_without_lca"
PYTEST_NO_SKIP=True ./run_nx_tests.sh --cov-append -k "TestDAGLCA and test_all_pairs_lca_pairs_without_lca"
PYTEST_NO_SKIP=True ./run_nx_tests.sh --cov-append -k "TestEfficiency and test_using_ego_graph"
# run_nx_tests.sh outputs coverage data, so check that total coverage is >0.0%
# in case nx-cugraph failed to load but fallback mode allowed the run to pass.
_coverage=$(coverage report|grep "^TOTAL")
Expand Down
11 changes: 0 additions & 11 deletions python/nx-cugraph/nx_cugraph/interface.py
Original file line number Diff line number Diff line change
Expand Up @@ -242,20 +242,9 @@ def key(testpath):
)

too_slow = "Too slow to run"
maybe_oom = "out of memory in CI"
skip = {
key("test_tree_isomorphism.py:test_positive"): too_slow,
key("test_tree_isomorphism.py:test_negative"): too_slow,
key("test_efficiency.py:TestEfficiency.test_using_ego_graph"): maybe_oom,
key("test_dag.py:TestDAG.test_antichains"): maybe_oom,
key(
"test_lowest_common_ancestors.py:"
"TestDAGLCA.test_all_pairs_lca_pairs_without_lca"
): maybe_oom,
key(
"test_lowest_common_ancestors.py:"
"TestMultiDiGraph_DAGLCA.test_all_pairs_lca_pairs_without_lca"
): maybe_oom,
# These repeatedly call `bfs_layers`, which converts the graph every call
key(
"test_vf2pp.py:TestGraphISOVF2pp.test_custom_graph2_different_labels"
Expand Down

0 comments on commit a5f4a72

Please sign in to comment.