-
Notifications
You must be signed in to change notification settings - Fork 1k
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.
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
Remove ECOS dependency #2301
Comments
Hi @pwithams! Until then, you can simply install |
as ecos wheel is not available for Python-3.12 on Windows, making it optional unless "pip install cxvpy[ecos]" is asked, would be helpfull. same for osqp, there shall be if possible a "pip install cxvpy[osqp]" |
@phschiele Did you mean to say 1.6 here? (I know the original commenter said 1.16.) |
Yes, it should have been 1.6, thanks! |
This feature is "planned" for 1.16 (https://www.cvxpy.org/updates/index.html#ecos-deprecation) so this issue can be viewed as a +1 for that plan and a request to expedite it if possible.
Is your feature request related to a problem? Please describe.
cvxpy
(apache license) currently installsecos
(gpl license) by default. However,ecos
is not actually required for somecxvpy
functionality and so avoiding installing gpl packages currently requires manually installing cherry pickedcvxpy
dependencies. For example, doing a regular install ofcvxpy
will fail apip-licenses
check for gpl due toecos
(https://github.com/raimon49/pip-licenses#option-fail-on).Describe the solution you'd like
Describe alternatives you've considered
cvxpy
is actually gpl and just not using the package, but this doesn't align with Switch from GPLv3 to Apache 2.0 #313Additional context
#313 mentions "cvxpy does not import the ecos interface or any other solver interface unless the user has installed that solver and tries to use it" which seems a prime candidate for optional install.
https://www.cvxpy.org/updates/index.html#ecos-deprecation expresses intent to do this exact change, just only in 1.16: " In 1.6, we plan to no longer install ECOS as a CVXPY dependency"
The text was updated successfully, but these errors were encountered: