Skip to content

Commit 8389510

Browse files
Transurgeonclaude
andcommitted
Keep IPOPT extra commented out to avoid --all-extras breakage
The test_optional_solvers workflow uses uv sync --all-extras, which would try to build cyipopt without system IPOPT installed. Instead, install cyipopt directly via uv pip in test_nlp_solvers where the system library is available. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent e28083f commit 8389510

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

.github/workflows/test_nlp_solvers.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,9 @@ jobs:
2929
if: runner.os == 'macOS'
3030
run: brew install ipopt
3131
- name: Install dependencies
32-
run: uv sync --extra IPOPT --dev
32+
run: |
33+
uv sync --dev
34+
uv pip install cyipopt
3335
- name: Print installed solvers
3436
run: uv run python -c "import cvxpy; print(cvxpy.installed_solvers())"
3537
- name: Run nlp tests

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ SCS = []
9494
XPRESS = ["xpress>=9.5"]
9595
DAQP = ["daqp"]
9696
KNITRO = ["knitro"]
97-
IPOPT = ["cyipopt"]
97+
# IPOPT = ["cyipopt"] # requires system IPOPT; excluded from --all-extras
9898
testing = ["pytest", "hypothesis"]
9999
doc = ["sphinx",
100100
"sphinxcontrib.jquery",

0 commit comments

Comments
 (0)