Skip to content

Commit b311a99

Browse files
ci(thirdparty-pulp): skip test_numpy_float in nightly cuOpt run (NVIDIA#1190)
## Summary PuLP's `test_numpy_float` calls `model.solve()` without an explicit solver and relies on PuLP's default-solver auto-detection. `CUOPT` is not on that default list, so in our cuopt-only nightly test environment the call raises `PulpError: No solver available` and the test fails for reasons unrelated to cuOpt. This PR deselects the test in `ci/thirdparty-testing/run_pulp_tests.sh` with an inline comment explaining the cause. ## Context — other cuOpt-related PuLP failures | Test | Status | |---|---| | `test_unbounded` | Fixed upstream by [coin-or/pulp#901](coin-or/pulp#901) (tracked in NVIDIA#1114) | | `test_integer_infeasible_2` | Fixed upstream by [coin-or/pulp#901](coin-or/pulp#901) (tracked in NVIDIA#1114) | | `test_infeasible_2` | Real cuOpt bug — PDLP returns Optimal for an infeasible LP. Tracked in NVIDIA#1115. Not skipped here. | | `test_numpy_float` | **This PR** | ## Test plan - [ ] Nightly third-party-PuLP job no longer reports `test_numpy_float` as failing - [ ] Other cuOpt PuLP tests still execute (no over-broad deselect) Authors: - Ramakrishnap (https://github.com/rgsl888prabhu) Approvers: - Miles Lubin (https://github.com/mlubin) - Trevor McKay (https://github.com/tmckayus) URL: NVIDIA#1190
1 parent f3ab099 commit b311a99

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

ci/thirdparty-testing/run_pulp_tests.sh

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,15 @@ rapids-logger "running PuLP tests (cuOpt-related)"
3030
# PuLP uses pytest; run only tests that reference cuopt/CUOPT
3131
# Exit code 5 = no tests collected; then try run_tests.py which detects solvers (including cuopt)
3232
pytest_rc=0
33+
# test_numpy_float calls model.solve() with no explicit solver; PuLP's
34+
# default-solver auto-detection list doesn't include CUOPT, so it raises
35+
# "No solver available" in our cuopt-only test environment. Skip it here.
3336
timeout 5m python -m pytest \
3437
--verbose \
3538
--capture=no \
3639
--junitxml="${RAPIDS_TESTS_DIR}/junit-thirdparty-pulp.xml" \
3740
-k "cuopt or CUOPT" \
41+
--deselect pulp/tests/test_pulp.py::CUOPTTest::test_numpy_float \
3842
pulp/tests/ || pytest_rc=$?
3943

4044
if [ "$pytest_rc" -eq 5 ]; then

0 commit comments

Comments
 (0)