-
Notifications
You must be signed in to change notification settings - Fork 140
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
Docplex is not fully compatible with cplex 20.1 #24
Comments
I can fix this as part of #23 |
We need to include a fix for Aqua 0.8.2 bugfix release. I suggest to pin docplex 2.15.194 for the release. What do you think? |
The PR #30 runs all unit tests under Ubuntu for all python versions, with and without cplex and cvxpy with no restrictions. |
As Manoel noted above, the unit tests are running with the latest docplex and all seem to be working fine without cplex being installed. Can we review the analysis that led to us pinning an older version in the recent stable release as to how to move forwards here? Are we missing some unit test that would show the failure that the review above hit - maybe its there on Mac but does not manifest in the platform used for the build? |
@manoelmarques @woodsp-ibm Thank you for the unit tests without cplex. I notice from the result that the issue of docplex without cplex occurs only on macOS. Docplex 2.20 works fine without cplex on Ubuntu and Windows. I updated the table. So, we need version pin of docplex is necessary for only macOS. |
@t-imamichi I see there is a cplex 20.1.0.1 that was released and says its from Feb 8th. Is it worth double checking the status of the docplex/cplex ahead of release to see if the version pinning we have in place still makes sense or whether it can be relaxed at all. |
Thank you for the notice. I tried them and confirmed that the two NGs in the table remain as of Mar. 26, 2021. |
This issue is resolved with docplex 2.21.207. This issue will be close by #155. |
Information
What is the current behavior?
There are two issues related to docplex as follows.
(1)
Docplex 2.14.186 (May 2020) or older does not work with cplex 20.1 (Jan 2021)
(2)
Docplex 2.16.196 (Nov 2020) or newer does not work without installing
cplex
. But, cplex is not included as the dependency of docplex. It seems a bug. Because nocplex
wheel available for Python 3.9, docplex 2.16.196 or newer does not work with Python 3.9.cplex
version also matters.This change is mentioned as follows.
Summary of compatibility:
Steps to reproduce the problem
Python 3.7, docplex 2.15.194 without cplex
Python 3.7, docplex 2.16.196 with cplex 20.1
Python 3.7, docplex 2.16.196 without cplex
Python 3.8, docplex 2.15.194 with and without cplex
Python 3.8, docplex 2.16.196 with cplex 20.1
Python 3.8, docplex 2.16.196 without cplex
Python 3.8, docplex 2.14.186 with cplex 20.1
Python 3.9, docplex 2.15.194 without cplex
Python 3.9, docplex 2.16.196 without cplex
What is the expected behavior?
QuadraticProgram
works fine.Suggested solutions
I recommend to pin 2.15.194 (Jul 2020) for the time being because it is the only docplex that works with any Python 3.6-3.9 even without cplex wheel.
The text was updated successfully, but these errors were encountered: