-
Notifications
You must be signed in to change notification settings - Fork 10
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
Feature/antares in memory #753
Merged
Merged
Changes from 116 commits
Commits
Show all changes
173 commits
Select commit
Hold shift + click to select a range
9348afe
Encapsulate RunProblemGeneration in a class
JasonMarechal25 a7bab62
Refactor
JasonMarechal25 f1be1a0
Deduce archive name from xpansion output name
JasonMarechal25 cfb8814
Handle not deductible archive file name
JasonMarechal25 757750e
Move output and archive deduction in ProblemGenerationExeOptions
JasonMarechal25 0408426
Only use archive for run
JasonMarechal25 281f918
Fix build
JasonMarechal25 468f38f
wip
JasonMarechal25 3f31f71
--study options for pbgen
JasonMarechal25 0a63126
Minor cleanup
JasonMarechal25 e6bba95
destructor
JasonMarechal25 5fc8007
Use Lps in pbGen
JasonMarechal25 ee8e3f0
Use variables name in problems from Antares
JasonMarechal25 9b90b6b
Use problemName from Antares
JasonMarechal25 ae98609
Use "mode"
JasonMarechal25 9c3732d
Refactor test
JasonMarechal25 7736cbf
Add study for lpNamer e2e tests
JasonMarechal25 5d13f0e
Wip full run in memory
JasonMarechal25 ac6fb50
API and file work in -xpansion properly
JasonMarechal25 1945cc2
It works E2E
JasonMarechal25 2e8772e
Reintroduce files
JasonMarechal25 3bb38c0
Reintroduce files
JasonMarechal25 3e0ae17
Fix output mode renaming file instead of copying
JasonMarechal25 002a255
Ensure zip is clean for tests
JasonMarechal25 9bb2eef
Fix tests
JasonMarechal25 cf05426
Fix
JasonMarechal25 6d71d50
Use constraints name
JasonMarechal25 e939699
Don't write mps
JasonMarechal25 0957d99
Change the correct number of var/col name
JasonMarechal25 86a30dd
Round to 10
JasonMarechal25 665fe74
Init solver created by factory
JasonMarechal25 2a7ff36
Cleanup
JasonMarechal25 32d5242
Study mode: Output in simuilation dir, not -Xpansion dir
JasonMarechal25 7c0a8d9
Read variable names in whebdo data
JasonMarechal25 44734a3
Fix LPNamer test archive
JasonMarechal25 01f1044
Rename area and interco files to match reality
JasonMarechal25 9c6c8ee
Use parametrize to generate test combination automatically
JasonMarechal25 e2c9a6c
Execute lpname e2e tests in tmp directories
JasonMarechal25 caa6890
tidy code
JasonMarechal25 c8cc927
Restore unamed problem
JasonMarechal25 fec196e
Fix LPNamer e2e tests
JasonMarechal25 3f0676f
ADR: Memory, archive and file mode support
JasonMarechal25 0ddc68b
ADR: Nominal case in memory
JasonMarechal25 88586e4
Fix tests
JasonMarechal25 d5eb74b
remove locker
JasonMarechal25 4ed9607
Format Cmake
JasonMarechal25 5b2bcac
Fix post rebase
JasonMarechal25 59aa662
Fix test
JasonMarechal25 5f37342
f
JasonMarechal25 695b18d
Fix test
JasonMarechal25 9ceff72
Don't use antares step in drivers in memory mode
JasonMarechal25 9ff43db
Extract complexe code in meaningful method
JasonMarechal25 cf6c3ad
Remove useless code
JasonMarechal25 cdb799e
Re-enable parallelisation
JasonMarechal25 61d01f2
Fix various typo
JasonMarechal25 9e64a73
Remove superfluous solver.init
JasonMarechal25 6dc1361
Typo
JasonMarechal25 5904571
Typo
JasonMarechal25 bc4c424
Complete ADR: Explain why archive mode is still supported despite acc…
JasonMarechal25 e75f746
C4 architecture diagrams (#736)
JasonMarechal25 c8e80d6
[Pb Gen] deal with last study not being a zip (#756)
a-zakir f283c0d
Release 1.2.2 (#759)
JasonMarechal25 bd871df
ADR subproblem data storage (#760)
tbittar b3a9936
xpress licence in custom path (#758)
a-zakir aa355e7
Enable dependabot (#763)
JasonMarechal25 05e0260
deliver json cpp (#762)
JasonMarechal25 871b627
Produce only one asset (with xpress) (#764)
JasonMarechal25 f68c8d4
Bump bruceadams/get-release from 1.2.3 to 1.3.2 (#769)
dependabot[bot] 71ba9ef
Bump actions/cache from 3 to 4 (#766)
dependabot[bot] 5ee9e93
Bump actions/setup-python from 3 to 5 (#765)
dependabot[bot] f86c10e
Bump actions/checkout from 3 to 4 (#767)
dependabot[bot] 662d028
Revert to checkout v3 for centOS (#772)
JasonMarechal25 e160302
[ANT-1224] Preparing loop around benders solver (#739)
a-zakir 14f192a
Add high level design information (#757)
JasonMarechal25 b8e0396
Use vcpkg for dependency management on all platforms (#770)
JasonMarechal25 b152610
Fix sonar (#773)
JasonMarechal25 ded1dab
fix single file name (#785)
a-zakir 9c2f1e2
fix xpress detection spam (#771)
a-zakir 3f8e6ce
Feature/embed mpi (#786)
a-zakir 3462b08
[ANT-1461] sensitivity study with unzipped outputs (#787)
a-zakir 6ef3d0e
Bump tj-actions/branch-names from 6 to 8 (#774)
dependabot[bot] d367b25
Encapsulate RunProblemGeneration in a class
JasonMarechal25 757cbe4
Wip full run in memory
JasonMarechal25 30c4303
It works E2E
JasonMarechal25 b3e3fa0
Cleanup
JasonMarechal25 9dc499e
Update antares targets
JasonMarechal25 1cc95aa
Update antares API
JasonMarechal25 5bef82e
Fix rebase
JasonMarechal25 e6de016
Fix missing file
JasonMarechal25 4c57e2f
Fix
JasonMarechal25 ee686f9
Fix missing files
JasonMarechal25 1772d5b
Merge remote-tracking branch 'origin/develop' into feature/antares_in…
JasonMarechal25 fb70f92
fix
JasonMarechal25 b1baccd
Remove file
JasonMarechal25 a793b0c
Better handling of mode with switch
JasonMarechal25 320fa39
Extract method
JasonMarechal25 b2a4906
Extract method
JasonMarechal25 e6c98bd
Document
JasonMarechal25 c2a9239
Extract method
JasonMarechal25 fbb3e8a
Extract method
JasonMarechal25 23c9ef7
Extract method
JasonMarechal25 8c263cd
Reorder
JasonMarechal25 63db1e3
Simplify
JasonMarechal25 9eca5d8
typo
JasonMarechal25 6b542c8
Remove unused code
JasonMarechal25 132e1fa
Merge remote-tracking branch 'refs/remotes/origin/develop' into featu…
JasonMarechal25 43a28b1
Merge branch 'develop' into feature/antares_in_memory
JasonMarechal25 0be3097
Improve ADR
JasonMarechal25 3901ebb
Merge branch 'develop' into feature/antares_in_memory
JasonMarechal25 8f705e3
[API]Use antares lib (#825)
JasonMarechal25 789c826
Merge branch 'develop' into feature/antares_in_memory
JasonMarechal25 f166355
Try fix build
JasonMarechal25 88b2e3f
Merge branch 'develop' into feature/antares_in_memory
JasonMarechal25 0411b69
Fix build
JasonMarechal25 463a9ea
Update antares to 9.2.0-rc1
JasonMarechal25 0a21b13
Remove temporary trick for antares versions
JasonMarechal25 2afb6d6
Update ortools
JasonMarechal25 000e061
Update ortools
JasonMarechal25 f24d9d9
Fix data
JasonMarechal25 6bd53fe
Remove sirius port
JasonMarechal25 1c28acc
Fix Link when coin is not build
JasonMarechal25 e66ebb7
Don't force static build of coin. Avoid incompatibility issue with or…
JasonMarechal25 64b39e6
Download precompiled sirius
JasonMarechal25 776f2a1
Download precompiled sirius
JasonMarechal25 2dba990
Fix assets
JasonMarechal25 d97aac0
Fix assets
JasonMarechal25 50ec079
Fix assets
JasonMarechal25 8fd8281
Fix assets
JasonMarechal25 679d6c3
Fix windows
JasonMarechal25 c142820
Fix oracle
JasonMarechal25 2c03d9c
Fix windows
JasonMarechal25 5da8233
Fix windows
JasonMarechal25 042bcfd
Merge branch 'develop' into feature/antares_in_memory
JasonMarechal25 791dfef
Force don't build antares to debug why not detected
JasonMarechal25 644156e
Revert data
JasonMarechal25 9f85c05
Merge remote-tracking branch 'origin/develop' into feature/antares_in…
JasonMarechal25 646263e
Update ortools version
JasonMarechal25 30f3122
Update ortools
JasonMarechal25 60eea94
Don't use ortools with sirius asset
JasonMarechal25 7540cfd
Merge branch 'develop' into feature/antares_in_memory
JasonMarechal25 66e95e7
f
JasonMarechal25 be01d60
Ortools pre-release
JasonMarechal25 522b67e
Merge branch 'develop' into feature/antares_in_memory
JasonMarechal25 361607b
Update ortools
JasonMarechal25 3939b19
Use devtoolset 11
JasonMarechal25 e548185
f
JasonMarechal25 b831f10
f
JasonMarechal25 e432a76
f
JasonMarechal25 8e77cbe
f
JasonMarechal25 801bfae
f
JasonMarechal25 63e94f2
Fix missing key
JasonMarechal25 899353e
Revert data modifications
JasonMarechal25 4b4de93
Naming improvement
JasonMarechal25 d7b2ae5
Merge branch 'develop' into feature/antares_in_memory
JasonMarechal25 481a183
Merge branch 'develop' into feature/antares_in_memory
JasonMarechal25 d59edd9
Remove extra file
JasonMarechal25 eab1d8a
Merge branch 'develop' into feature/antares_in_memory
JasonMarechal25 276c05b
Fix merge
JasonMarechal25 4026911
Remove roundToTen
JasonMarechal25 c5748ae
Update ADR
JasonMarechal25 057eb22
Fix typo run-as-root
JasonMarechal25 1800dff
Merge branch 'develop' into feature/antares_in_memory
JasonMarechal25 f79826a
Test
JasonMarechal25 ea827ef
Updater xpress ref
JasonMarechal25 27e924a
Update reference mps
JasonMarechal25 5bcebc0
Merge branch 'develop' into feature/antares_in_memory
JasonMarechal25 f59ab8f
Better handling of SimulationMode unset value
JasonMarechal25 07bfd5e
Better error handling
JasonMarechal25 01caff5
Improve getXpansionProblems
JasonMarechal25 be03ffd
Simplification
JasonMarechal25 264b261
Fix build
JasonMarechal25 860c2e1
Merge branch 'develop' into feature/antares_in_memory
JasonMarechal25 7b07871
Fix build
JasonMarechal25 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -82,6 +82,7 @@ jobs: | |
XPRESS_CONTAINER: ${GITHUB_WORKSPACE}/xpress/bin | ||
XPRS_LIB_Path_CONTAINER: ${GITHUB_WORKSPACE}/xpress/lib | ||
VCPKG_ROOT: ${{ github.workspace }}/vcpkg | ||
ORTOOLS_DIR: $GITHUB_WORKSPACE/or-tools | ||
|
||
outputs: | ||
TGZ_NAME: ${{ steps.export_output.outputs.TGZ_NAME }} | ||
|
@@ -95,7 +96,11 @@ jobs: | |
if: github.event_name == 'release' && github.event.action == 'created' | ||
id: get_release | ||
uses: bruceadams/[email protected] | ||
|
||
|
||
- name: Config OR-Tools URL | ||
run: | | ||
echo "ORTOOLS_URL=https://github.com/rte-france/or-tools/releases/download/v9.8-rte1.0/ortools_cxx_centos7_static_sirius.zip" >> $GITHUB_ENV | ||
|
||
- uses: actions/checkout@v3 #Keep at 3 | ||
with: | ||
submodules: true | ||
|
@@ -124,7 +129,9 @@ jobs: | |
antares-version: ${{needs.versions.outputs.antares-version}} | ||
os: centos7 | ||
os-full-name: CentOS-7.9.2009 | ||
|
||
ortools-url: ${{env.ORTOOLS_URL}} | ||
ortools-dir: ${{env.ORTOOLS_DIR}} | ||
|
||
- name: repo update | ||
run: | | ||
yum update | ||
|
@@ -164,11 +171,12 @@ jobs: | |
source /opt/rh/devtoolset-10/enable | ||
cmake -B _build -S . \ | ||
-DBUILD_TESTING=ON \ | ||
-DCMAKE_PREFIX_PATH=${GITHUB_WORKSPACE}/deps \ | ||
-DCMAKE_PREFIX_PATH="${GITHUB_WORKSPACE}/deps;${{env.ORTOOLS_DIR}}/install" \ | ||
-DCMAKE_BUILD_TYPE=Release \ | ||
-DCMAKE_INSTALL_PREFIX=_install \ | ||
-DBUILD_UI=ON \ | ||
-DALLOW_RUN_AS_ROOT=ON \ | ||
-DVCPKG_TARGET_TRIPLET=x64-linux-release \ | ||
-DCMAKE_TOOLCHAIN_FILE=vcpkg/scripts/buildsystems/vcpkg.cmake | ||
|
||
- name: Build | ||
|
@@ -187,7 +195,7 @@ jobs: | |
timeout-minutes: 120 | ||
shell: bash | ||
run: | | ||
export PATH=${GITHUB_WORKSPACE}/_build/vcpkg_installed/x64-linux/tools/openmpi/bin/:$PATH | ||
export PATH=${GITHUB_WORKSPACE}/_build/vcpkg_installed/x64-linux-release/tools/openmpi/bin/:$PATH | ||
export LD_LIBRARY_PATH=LD_LIBRARY_PATH:${{ env.XPRS_LIB_Path_CONTAINER }} | ||
export XPRESS=${{ env.XPRESS_CONTAINER }} | ||
cd _build | ||
|
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
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.
to be updated check that --> message