-
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
Add mps/gz support #582
Comments
Users can import and export docplex models via |
As commented in #581 (comment), I personally suggest making functions in |
Sure, we can move it there @t-imamichi . In this case, I'd vote that we also move all Since this breaks the |
For breaking API changes maintaining compatibility via deprecation, which informs the replacement (or whatever) to maintain users code working and give them a chance to change, is definitely the way to go. Works for me. The deprecation would also be noted in a release note As the move will involve a few test files that use the logic having the move/deprecation done as a separate PR could also be a choice. |
@woodsp-ibm Sounds good. I'll proceed then as follows:
|
What should we add?
What should we add?
Most collections of optimization models use the old-fashioned MPS format for model storage, e.g., MIPLIB. Furthermore, due to their verbosity (compare, e.g., square47.mps.gz wit squaren47.mps from MIPLIB - 80 MB vs. 1.4GB) people tend to use the gzipped files directly.
If qiskit optimization serves as gateway to classical and quantum optimization, I think it makes perfect sense to include this widely-adopted file format. docplex supports mps natively.
The text was updated successfully, but these errors were encountered: