Skip to content

Fix typos and cleanup tests#1

Open
SteveDiamond wants to merge 1 commit intomasterfrom
codex/identify-and-propose-codebase-improvements
Open

Fix typos and cleanup tests#1
SteveDiamond wants to merge 1 commit intomasterfrom
codex/identify-and-propose-codebase-improvements

Conversation

@SteveDiamond
Copy link
Owner

Summary

  • fix typo in Mosek solver comment
  • fix typo in test_conic_solvers
  • add missing space in PowCone3D error message
  • correct "AddExpression" spelling in tests
  • remove stray prints in test_complex

Testing

  • ruff check . (fails: Found 38 errors)
  • pytest cvxpy/tests/test_complex.py -k test_validation -q (fails: ImportError: numpy.core.multiarray failed to import)

https://chatgpt.com/codex/tasks/task_e_68437cbb14648331bbc0dc2a0a237f73

SteveDiamond pushed a commit that referenced this pull request Feb 23, 2026
* Add Knitro interface to CVXPY for Quadratic Problems and Conic Problems (#1)

* Add KNITRO solver to the list of available solvers

* Add KNITRO QP and Conic solver interfaces

* Add KNITRO citation to the citation dictionary

* Add KNITRO QP and Conic solver interface implementation

* Add KNITRO to doc

* Add KNITRO to ci for optional solvers

* Use KNITRO_LICENSE instead of KNITRO_CI.

* Fix KNITRO installation condition for macOS architecture check

* Update copyright in conic_solver.py and qp_solver.py

* Update copyright information

* Reverse back the copyright

* Remove unused import statement for knitro in KNITRO solver classes

* Add import statement for Knitro solver in KNITRO classes

* Remove unused import statement for Knitro in KNITRO solver classes

* Add documentation reference for Knitro status mapping in KNITRO solver classes

* Fix formatting of KNITRO license echo command in CI workflow

---------

Co-authored-by: William Zijie Zhang <89562186+Transurgeon@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant