Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Delete CPP backend #63

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Delete CPP backend #63

wants to merge 4 commits into from

Conversation

Transurgeon
Copy link
Owner

Description

Just testing if this would work since we are starting to explore new options (Cython and Rust).
All tests seem to be passing even when deleting Sparse Cholesky.
Issue link (if applicable):

Type of change

  • New feature (backwards compatible)
  • New feature (breaking API changes)
  • Bug fix
  • Other (Documentation, CI, ...)

Contribution checklist

  • Add our license to new files.
  • Check that your code adheres to our coding style.
  • Write unittests.
  • Run the unittests and check that they’re passing.
  • Run the benchmarks to make sure your change doesn’t introduce a regression.

bkurtz and others added 4 commits October 15, 2024 12:05
@bkurtz
Re-enable sdist publishing + clean up build workflows (cvxpy#2585)
01e11e4 
* combine base/non-base configs

* better matrix handling

* clean up matrix entries

* clean up stray double spaces

* include openmp build/test config

* clearer name plus commenta bout building cvxpy-base

* add comments about matrix include

* need pytest-cov

* test matrix entries as booleans

* cplex doesn't have 3.12 wheels yet

* set BASE suffix in a way that'll be easier to see is correct before merging

* revertme: test condition for building sdist

* wrong string quotes

* Revert "revertme: test condition for building sdist"

This reverts commit 9d53631.

* use simple boolean on second half too
@bkurtz
Fix artifact upload + clean up after local pip install (cvxpy#2590)
75ac76c
@Transurgeon
delete CPP backend for 2.0
dd30c7a
delete NumPyCanonBackend for cleaning up the canon_backend.py file
7c6d8f1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Transurgeon @bkurtz