-
Notifications
You must be signed in to change notification settings - Fork 24
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 optimization options in OR-Tools / XPRESS, refactor #1837
Merged
Merged
Changes from 31 commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
e7c21b0
Version 8.8.0-rc4
flomnes 83dc5b5
Add solver parameters from command line
tbittar 06f6fbd
Run CI on CentOS7
flomnes dce2220
Merge remote-tracking branch 'origin/develop' into fix/xpress-perf
flomnes ccba98f
Update URL
flomnes 3178830
Merge remote-tracking branch 'github/develop' into fix/xpress-perf
flomnes 0707458
Merge remote-tracking branch 'github/develop' into fix/xpress-perf
flomnes 12c11ff
Reformat + use OptimizationOptions
flomnes 23f67dc
Add CMake target for Optimization options
flomnes 1657b29
Merge branch 'develop' into fix/xpress-perf
tbittar a79dc3a
Merge remote-tracking branch 'origin/develop' into fix/xpress-perf
tbittar 56a1cd6
Fix conflicts after rebase
tbittar 39492c8
Set generic parameters from generaldata.ini
tbittar e5ef606
Fix typo
tbittar 7808ea9
Set solver parameters only once and closer to solve
tbittar e25baf6
Add some const qualifiers
tbittar 3aca9c2
Gather all solverLogs and ortools options in optimization options
tbittar 59e60a7
Remove use basis config
tbittar 5d2beb6
Merge branch 'develop' into fix/xpress-perf
flomnes fa8ee43
Add solver specific parameters check
tbittar fe6add0
Add unit tests
tbittar 7ea6f86
Merge branch 'develop' into fix/xpress-perf
tbittar 9ef4d87
Clarify tests and free solver when not needed anymore
tbittar 1c084a8
Add docs
tbittar b7da5c3
update tests
tbittar 28b4e82
Fix tests
tbittar db4b662
Take into account (almost all) PR remarks
tbittar 30dc39f
Fix cmake
tbittar fb2e8b6
Perform specific parameters check only in solve
tbittar 5afadd1
Restore command line option management
tbittar 2a7154f
Allow to set solver parameters from command line, define xpress and s…
tbittar 54ef5e4
Merge branch 'develop' into fix/xpress-perf
tbittar 1282466
Differentiate between xpress and scip parameters
tbittar 2d7cefc
Factorize code
tbittar bb5e9a9
Merge branch 'develop' into fix/xpress-perf
tbittar 2f03618
Update docs
tbittar 166b261
Fix bugs from previous merge
tbittar a5aecb2
Fix bug
tbittar b8a1128
Merge branch 'develop' into fix/xpress-perf
tbittar c563243
Fix typo
tbittar 2c8d2df
remove unused declarations
tbittar a124741
Simplify spec (remove generaldata.ini new stuff, unify CLI options)
flomnes 49a30ba
Fix changelog
flomnes dca22ca
f
flomnes File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -464,4 +464,4 @@ endif() | |
include(CPack) | ||
|
||
#Last. We need all target defined | ||
add_subdirectory(packaging) | ||
# add_subdirectory(packaging) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Revert this |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
set(PROJ optimization-options) | ||
|
||
add_library(${PROJ} INTERFACE) | ||
|
||
target_include_directories(${PROJ} | ||
INTERFACE | ||
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include> | ||
) | ||
|
||
add_library(Antares::${PROJ} ALIAS ${PROJ}) |
34 changes: 34 additions & 0 deletions
34
src/libs/antares/optimization-options/include/antares/optimization-options/options.h
pet-mit marked this conversation as resolved.
Show resolved
Hide resolved
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,34 @@ | ||
/* | ||
** Copyright 2007-2024, RTE (https://www.rte-france.com) | ||
** See AUTHORS.txt | ||
** SPDX-License-Identifier: MPL-2.0 | ||
** This file is part of Antares-Simulator, | ||
** Adequacy and Performance assessment for interconnected energy networks. | ||
** | ||
** Antares_Simulator is free software: you can redistribute it and/or modify | ||
** it under the terms of the Mozilla Public Licence 2.0 as published by | ||
** the Mozilla Foundation, either version 2 of the License, or | ||
** (at your option) any later version. | ||
** | ||
** Antares_Simulator is distributed in the hope that it will be useful, | ||
** but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
** MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
** Mozilla Public Licence 2.0 for more details. | ||
** | ||
** You should have received a copy of the Mozilla Public Licence 2.0 | ||
** along with Antares_Simulator. If not, see <https://opensource.org/license/mpl-2-0/>. | ||
*/ | ||
|
||
#pragma once | ||
|
||
namespace Antares::Solver::Optimization { | ||
struct OptimizationOptions | ||
{ | ||
//! Force ortools use | ||
bool ortoolsUsed = false; | ||
//! The solver name, sirius is the default | ||
std::string ortoolsSolver = "sirius"; | ||
bool solverLogs = false; | ||
std::string solverParameters = ""; | ||
}; | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is markdown, not HTML. Please remove markups and use the markdown syntax instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A bullet list inside a table does not render well if I use the markdown syntax, therefore I switch to html for this particular feature: https://stackoverflow.com/questions/19950648/how-to-write-lists-inside-a-markdown-table It renders well on the readthedocs.
Does this have an impact elsewhere ? PDF generation maybe ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should try to avoid html as much as possible too