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.

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

use XPRSmipoptimize and XPRSlpoptimize instead of XPRSminim and XPRSmaxim #114

Merged

Conversation

sgatto
Copy link
Member

@sgatto sgatto commented Nov 16, 2023

XPRSminim and XPRSmaximare deprecated see xpress doc
I propose to use XPRSlpoptimize and XPRSmipoptimize as suggested by the doc

@sgatto sgatto requested review from flomnes and pet-mit November 16, 2023 09:57
Copy link
Collaborator

@pet-mit pet-mit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

very important: you should also call XPRSchgobjsense before calling XPRSmipoptimize or XPRSlpoptimize, it is currently only called in the constructor (the user can change the optimization sense at any time by using MPObjective::SetMinimization/Maximization)

ortools/linear_solver/xpress_interface.cc Outdated Show resolved Hide resolved
ortools/linear_solver/xpress_interface.cc Outdated Show resolved Hide resolved
@sgatto sgatto merged commit 0a08de7 into feature/xpress_only_RTE Nov 16, 2023
sgatto added a commit that referenced this pull request Nov 17, 2023
…axim (#114)

* use XPRSmipoptimize and XPRSlpoptimize instead of XPRSminim and XPRSmaxim

* clean xpress/environment files

* accept changes: empty char* parameter for XPRS*optimize
flomnes added a commit that referenced this pull request Nov 20, 2023
* remove python script

* remove RTE actions

* fix test_xpress_interface.cc

* remove callback_xpress.py

* revert writing colnames and rownames

* accept suggestion from Mizux

* clean

* change cmake/README.md

* try fix build bazel

* try fix build bazel add MPSWriteError.h

* xpress tests gracefully exit if Xpress not found

* add integer and linear programming test for dotnet python and java

* remove MPSWriteError

* try fix Window build

* remove useless line from CMakeLists.txt

* try fix test under windows

* reformat

* use XPRESS_LP instead of XPRESS for linear programming examples

* tools: add --platform arg when possible

make script more resilient/cross-platform

* [CP-SAT] convert to PEP8 convention

* use XPRSmipoptimize and XPRSlpoptimize instead of XPRSminim and XPRSmaxim (#114)

* use XPRSmipoptimize and XPRSlpoptimize instead of XPRSminim and XPRSmaxim

* clean xpress/environment files

* accept changes: empty char* parameter for XPRS*optimize

* Add test on number iterations with LP basis

* fix gtests flags

* refactor

* suggestions by @flomnes

* remove unwanted files

---------

Co-authored-by: Andrea Sgattoni <[email protected]>
Co-authored-by: Laurent Perron <[email protected]>
Co-authored-by: Corentin Le Molgat <[email protected]>
Co-authored-by: Andrea Sgattoni <[email protected]>
flomnes added a commit that referenced this pull request Nov 20, 2023
* remove python script

* remove RTE actions

* fix test_xpress_interface.cc

* remove callback_xpress.py

* revert writing colnames and rownames

* accept suggestion from Mizux

* clean

* change cmake/README.md

* try fix build bazel

* try fix build bazel add MPSWriteError.h

* xpress tests gracefully exit if Xpress not found

* add integer and linear programming test for dotnet python and java

* remove MPSWriteError

* try fix Window build

* remove useless line from CMakeLists.txt

* try fix test under windows

* reformat

* use XPRESS_LP instead of XPRESS for linear programming examples

* tools: add --platform arg when possible

make script more resilient/cross-platform

* [CP-SAT] convert to PEP8 convention

* use XPRSmipoptimize and XPRSlpoptimize instead of XPRSminim and XPRSmaxim (#114)

* use XPRSmipoptimize and XPRSlpoptimize instead of XPRSminim and XPRSmaxim

* clean xpress/environment files

* accept changes: empty char* parameter for XPRS*optimize

* Add test on number iterations with LP basis

* fix gtests flags

* refactor

* suggestions by @flomnes

* remove unwanted files

---------

Co-authored-by: Andrea Sgattoni <[email protected]>
Co-authored-by: Laurent Perron <[email protected]>
Co-authored-by: Corentin Le Molgat <[email protected]>
Co-authored-by: Andrea Sgattoni <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants