Skip to content

feature(Time-Coupled): add LazyNetwork implementation#1644

Merged
Godelaine merged 9 commits intoci/core-7.2.0-SNAPSHOTfrom
feature/lazy-network
Mar 19, 2026
Merged

feature(Time-Coupled): add LazyNetwork implementation#1644
Godelaine merged 9 commits intoci/core-7.2.0-SNAPSHOTfrom
feature/lazy-network

Conversation

@bqth29
Copy link
Copy Markdown
Collaborator

@bqth29 bqth29 commented Mar 4, 2026

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • A PR or issue has been opened in all impacted repositories (if any)

Does this PR already have an issue describing the problem?

What kind of change does this PR introduce?

Creation of the LazyNetwork implementation that is based on a file path. The network data is only loaded when some data needs to be accessed. This allows to delete the RaoInputWithNetworkPaths API.

What is the current behavior?

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change or deprecate an API?

  • Yes
  • No

If yes, please check if the following requirements are fulfilled

  • The Breaking Change or Deprecated label has been added
  • The migration steps are described in the following section

What changes might users need to make in their application due to this PR? (migration steps)

Other information:

For this to work properly, we'll have to wait for this PR to be merged in powsybl-core.

bqth29 added 4 commits March 2, 2026 15:05
@bqth29
add lazy network implementation
b18ee6b 
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
@bqth29
fix all tests but cucumbers
528b0fd 
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
@bqth29
fix cucumbers
6a9c798 
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
@bqth29
Merge branch 'main' into feature/lazy-network
559f67b
@bqth29 bqth29 added PR: waiting-for-review This PR is waiting to be reviewed PR : pending-for-dependency-release This PR can't be merged until one dependency is updated breaking-change Changes could break users' code PR : wait-for-requirements This issue cannot be addressed for the moment as new features must be added to the code first labels Mar 4, 2026
@bqth29 bqth29 requested a review from phiedw March 4, 2026 15:09
bqth29 added 2 commits March 12, 2026 10:51
@bqth29
Merge branch 'main' into feature/lazy-network
014b6d8 
# Conflicts:
#	data/crac/crac-util/src/main/java/com/powsybl/openrao/data/crac/util/IcsImporter.java
#	data/crac/crac-util/src/test/java/com/powsybl/openrao/data/crac/util/IcsImporterTest.java
#	pom.xml
#	ra-optimisation/search-tree-rao/src/main/java/com/powsybl/openrao/searchtreerao/marmot/Marmot.java
#	ra-optimisation/search-tree-rao/src/test/java/com/powsybl/openrao/searchtreerao/marmot/MarmotTest.java
#	tests/src/test/java/com/powsybl/openrao/tests/steps/TimeCoupledRaoSteps.java
@bqth29
merge main
2131f68 
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
@bqth29 bqth29 changed the base branch from main to ci/core-7.2.0-SNAPSHOT March 13, 2026 14:13
@bqth29
use core snapshot version
5fa0aa3 
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
@phiedw phiedw closed this Mar 17, 2026
@phiedw phiedw deleted the feature/lazy-network branch March 17, 2026 16:29
@bqth29 bqth29 restored the feature/lazy-network branch March 17, 2026 16:37
@bqth29 bqth29 reopened this Mar 17, 2026
@bqth29
use lazy networks in ics importer tests
8d60994 
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
Godelaine
Godelaine previously approved these changes Mar 19, 2026
View reviewed changes
@Godelaine Godelaine marked this pull request as ready for review March 19, 2026 13:47
@Godelaine
Merge branch 'ci/core-7.2.0-SNAPSHOT' into feature/lazy-network
800cacf 
Signed-off-by: Godelaine <godelaine.demontmorillon@rte-france.com>
@Godelaine Godelaine merged commit dc96ff0 into ci/core-7.2.0-SNAPSHOT Mar 19, 2026
5 of 11 checks passed
bqth29 added a commit that referenced this pull request Mar 23, 2026
@bqth29 @Godelaine
feature(Time-Coupled): add LazyNetwork implementation (#1644)
4999cbb 
* add lazy network implementation

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* fix all tests but cucumbers

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* fix cucumbers

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* merge main

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* use core snapshot version

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* use lazy networks in ics importer tests

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

---------

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
Signed-off-by: Godelaine <godelaine.demontmorillon@rte-france.com>
Co-authored-by: Godelaine <godelaine.demontmorillon@rte-france.com>
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
bqth29 added a commit that referenced this pull request Mar 23, 2026
@bqth29 @Godelaine
feature(Time-Coupled): add LazyNetwork implementation (#1644)
976a2fc 
* add lazy network implementation

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* fix all tests but cucumbers

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* fix cucumbers

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* merge main

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* use core snapshot version

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* use lazy networks in ics importer tests

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

---------

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
Signed-off-by: Godelaine <godelaine.demontmorillon@rte-france.com>
Co-authored-by: Godelaine <godelaine.demontmorillon@rte-france.com>
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
RoxaneChen02 added a commit that referenced this pull request Mar 31, 2026
@bqth29 @Godelaine
@olperr1 @RoxaneChen02
Integration/core 7.2.0 (#1670)
8eb0cb1 
* chore(CI/CD): Rename dangling lines to boundary lines and update JSON CRAC version to 2.10 (#1655)

* rename Dangling Line to Boundary Line

* bump JSON CRAC version to 2.10

* Adapt to new sensi API

---------

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
Co-authored-by: Godelaine de Montmorillon <godelaine.demontmorillon@rte-france.com>
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* feature(Time-Coupled): add `LazyNetwork` implementation (#1644)

* add lazy network implementation

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* fix all tests but cucumbers

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* fix cucumbers

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* merge main

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* use core snapshot version

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* use lazy networks in ics importer tests

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

---------

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
Signed-off-by: Godelaine <godelaine.demontmorillon@rte-france.com>
Co-authored-by: Godelaine <godelaine.demontmorillon@rte-france.com>
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* Fix unit tests after new OLF parameters addition

Signed-off-by: Olivier Perrin <olivier.perrin@rte-france.com>
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* Fix remaining 'DanglingLine' (renamed in 'BoundaryLine')

Signed-off-by: Olivier Perrin <olivier.perrin@rte-france.com>
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* remove ill defined time coupled rao input

Signed-off-by: Godelaine de Montmorillon <godelaine.demontmorillon@rte-france.com>
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

* update pom

Signed-off-by: Godelaine de Montmorillon <godelaine.demontmorillon@rte-france.com>

* update pom with olf release

Signed-off-by: Godelaine de Montmorillon <godelaine.demontmorillon@rte-france.com>

* update powsybl.entsoe.version to 3.2.0

Signed-off-by: CHEN Roxane <roxane.chen@rte-france.com>

* add tests to cover lazy network

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>

---------

Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
Signed-off-by: Godelaine <godelaine.demontmorillon@rte-france.com>
Signed-off-by: Olivier Perrin <olivier.perrin@rte-france.com>
Signed-off-by: Godelaine de Montmorillon <godelaine.demontmorillon@rte-france.com>
Signed-off-by: CHEN Roxane <roxane.chen@rte-france.com>
Co-authored-by: Godelaine <godelaine.demontmorillon@rte-france.com>
Co-authored-by: Olivier Perrin <olivier.perrin@rte-france.com>
Co-authored-by: CHEN Roxane <roxane.chen@rte-france.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@RoxaneChen02 RoxaneChen02 RoxaneChen02 approved these changes

@Godelaine Godelaine Godelaine left review comments

@phiedw phiedw Awaiting requested review from phiedw

Assignees

No one assigned

Labels

breaking-change Changes could break users' code PR : pending-for-dependency-release This PR can't be merged until one dependency is updated PR : wait-for-requirements This issue cannot be addressed for the moment as new features must be added to the code first PR: waiting-for-review This PR is waiting to be reviewed release-03/26

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@bqth29 @Godelaine @RoxaneChen02 @phiedw