-
Notifications
You must be signed in to change notification settings - Fork 10
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Introduce "study mode". A way to run xpansion and problem generation with a study folder as parameter. In this case Antares will not be run as a stand-alone but as a library by problem generation. - New ADRs - Add a "new" data set for lpnamer E2E tests: SmallTestFiveCandidates, a copy of the example provided by Xpansion. - Refactor test_lpnamerEndToEnd.py : when possible prefer the use multiple @parametrize instead of writing the combinations in a list. - Refactor test_lpnamerEndToEnd.py : copy the test data in a temporary folder and works there. Prevent creating outputs in source code. - Add a test case for Study mode in test_lpnamerEndToEnd.py - Add a new E2E test: short memory - Refactor ProblemGenerationExeOptionsTest to test several combinations of mutually exclusive parameters - In SolverFactory.cpp call Init() on solver objects before returning them. Prevent temporal coupling for Xpress solver where a call to init() was mandatory. - Add Antares as a build dependency Missing pieces (every thing is open to discussion): - [ ] Handle Antares error #888 - [x] Update general data ini - [ ] Revert general data ini #889 - [x] Write ADR - [ ] Update sequence diagram #890 - [ ] Maybe update C4 #890 - [x] Use enum mode in LPFileExtractor #797 - [ ] find/define a way to log before lp dir exist (for antares error for exemple) #888 - [ ] Handle variables.txt directly at Lps level (SignificationMetierDesVariables) - [ ] Factorise variables name and "calculate" time step with week number on xpansion side (A faire dans Antares bien et vite) - [x] Cleanup python to only support memory mode (See ADR, we keep some support for now) - [x] antares named problèmes force mps writing. Peut etre pas besoin de l'option named sachant que les variables sont nommées dans la structure LP - [ ] Mettre à jour les docs: notamment comment build, dépendances, etc #893 - [ ] Investigate to use full run executable with --study option and use fullrun driver #892 - [x] Handle PBGen step with memory - [ ] Add entry in multisolver to rename a range of col/var #753 (comment) #891 - [ ] Run Antares in parallele mode Following other PR: - [ ] Don't overly rename problem in API mode --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: abdoulbari zakir <[email protected]> Co-authored-by: tbittar <[email protected]> Co-authored-by: Thomas Bittar <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
- Loading branch information
1 parent
1280d2c
commit 1cf286a
Showing
53 changed files
with
4,718 additions
and
4,386 deletions.
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 |
---|---|---|
|
@@ -70,6 +70,7 @@ jobs: | |
container: 'antaresrte/xpansion-centos7:1.1.0' | ||
env: | ||
VCPKG_ROOT: ${{ github.workspace }}/vcpkg | ||
ORTOOLS_DIR: $GITHUB_WORKSPACE/or-tools | ||
|
||
outputs: | ||
TGZ_NAME: ${{ steps.export_output.outputs.TGZ_NAME }} | ||
|
@@ -83,7 +84,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-rte/releases/download/v9.10-rte1.2-test-4/ortools_cxx_centos7_static_sirius.zip" >> $GITHUB_ENV | ||
- uses: actions/checkout@v3 #Keep at 3 | ||
with: | ||
submodules: true | ||
|
@@ -102,7 +107,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 | ||
|
@@ -155,7 +162,7 @@ jobs: | |
source /opt/rh/devtoolset-11/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 \ | ||
|
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.