Conversation
added 5 commits
February 11, 2026 14:50
- No support of multi-component computation (no way to report if a base simulation succeeds on one component but fails on another) - No indication of the cause of a failure to compute a contingency, compared to what is reported for a loadflow run. It makes it hard to troubleshoot issues when results are missing for a contingency. - No indication whether the result is complete or only partial, if the operation has been canceled and the thread interrupted. - Enable to add multi-component support for sensitivity analysis inOpenLoadFlow - Enable to return partial results in case of interruption - Enable a better diagnosis of contingency failure Signed-off-by: Fabrice Buscaylet <fabrice.buscaylet@artelys.com>
- No support of multi-component computation (no way to report if a base simulation succeeds on one component but fails on another) - No indication of the cause of a failure to compute a contingency, compared to what is reported for a loadflow run. It makes it hard to troubleshoot issues when results are missing for a contingency. - No indication whether the result is complete or only partial, if the operation has been canceled and the thread interrupted. - Enable to add multi-component support for sensitivity analysis inOpenLoadFlow - Enable to return partial results in case of interruption - Enable a better diagnosis of contingency failure Signed-off-by: Fabrice Buscaylet <fabrice.buscaylet@artelys.com>
Signed-off-by: Fabrice Buscaylet <fabrice.buscaylet@artelys.com>
Signed-off-by: Fabrice Buscaylet <fabrice.buscaylet@artelys.com>
Signed-off-by: Fabrice Buscaylet <fabrice.buscaylet@artelys.com>
snoopfab
changed the title
added 3 commits
February 24, 2026 12:12
…tatus in precontingency case Signed-off-by: Fabrice Buscaylet <fabrice.buscaylet@artelys.com>
…r herited classes added tests coverage Signed-off-by: Fabrice Buscaylet <fabrice.buscaylet@artelys.com>
…rTest testTaskInterruption Signed-off-by: Fabrice Buscaylet <fabrice.buscaylet@artelys.com>
|
alicecaron
reviewed
Mar 4, 2026
|
|
||
| @Override | ||
| public void computationComplete() { | ||
| // WHAT TO DO HERE ?? |
Contributor
There was a problem hiding this comment.
set to computationComplete to true like for the json writer?
Contributor
Author
There was a problem hiding this comment.
Hello Alice.
The goal of the writer is to write the information on several csv files (unlike the json one that can handle all information in one single file). I'm a little bit reluctant to create a computation.csv with just a computationComplete column with a true value in one line hence my question. Maybe adding some other global statistics on the computation for the future in the computationComplete( Map<String,Double> computationStats) would have sense in a future PR ?
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Please check if the PR fulfills these requirements
Does this PR already have an issue describing the problem?
Fixes #3626
What kind of change does this PR introduce?
Feature:
Enable to add multi-component support for sensitivity analysis inOpenLoadFlow
Enable to return partial results in case of interruption
Enable a better diagnosis of contingency failure
What is the current behavior?
See #3626
What is the new behavior (if this is a feature change)?
Implemented changes described in #3626
Does this PR introduce a breaking change or deprecate an API?
If yes, please check if the following requirements are fulfilled
What changes might users need to make in their application due to this PR? (migration steps)
TODO
Other information: