-
Notifications
You must be signed in to change notification settings - Fork 4
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 isSecure for RaoResult #869
Conversation
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
Signed-off-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
Signed-off-by: Sébastien Murgey <sebastien.murgey@rte-france.com>
...sult/rao-result-impl/src/main/java/com/powsybl/openrao/data/raoresultimpl/RaoResultImpl.java
Outdated
Show resolved
Hide resolved
...-exporter/swe-cne-exporter/src/main/java/com/powsybl/openrao/data/swecneexporter/SweCne.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Sébastien Murgey <sebastien.murgey@rte-france.com>
Signed-off-by: Sébastien Murgey <sebastien.murgey@rte-france.com>
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.
Please update the documentation here: https://github.com/farao-community/farao-community.github.io/blob/master/pages/docs/output-data/rao-result/rao-result-json.md
i realize this is more complicated than it seemed. we may want to do some refactoring in the future. it would be nice to have a review from @phiedw too
...rao-result/rao-result-api/src/main/java/com/powsybl/openrao/data/raoresultapi/RaoResult.java
Outdated
Show resolved
Hide resolved
...rao-result/rao-result-api/src/main/java/com/powsybl/openrao/data/raoresultapi/RaoResult.java
Outdated
Show resolved
Hide resolved
...rao-result/rao-result-api/src/main/java/com/powsybl/openrao/data/raoresultapi/RaoResult.java
Outdated
Show resolved
Hide resolved
...sult/rao-result-impl/src/main/java/com/powsybl/openrao/data/raoresultimpl/RaoResultImpl.java
Outdated
Show resolved
Hide resolved
...sult/rao-result-impl/src/main/java/com/powsybl/openrao/data/raoresultimpl/RaoResultImpl.java
Outdated
Show resolved
Hide resolved
...sult/rao-result-impl/src/main/java/com/powsybl/openrao/data/raoresultimpl/RaoResultImpl.java
Outdated
Show resolved
Hide resolved
return false; | ||
} | ||
if (Arrays.stream(u).toList().contains(PhysicalParameter.FLOW)) { | ||
return optimizedFlowCnecs.stream() |
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.
use functional cost here also
.../java/com/powsybl/openrao/searchtreerao/result/impl/PreventiveAndCurativesRaoResultImpl.java
Outdated
Show resolved
Hide resolved
|
||
@Override | ||
public boolean isSecure(Instant optimizedInstant, PhysicalParameter... u) { | ||
if (ComputationStatus.FAILURE.equals(getComputationStatus())) { |
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.
also check computation status map
Signed-off-by: Sébastien Murgey <sebastien.murgey@rte-france.com>
Signed-off-by: Sébastien Murgey <sebastien.murgey@rte-france.com>
Signed-off-by: Sébastien Murgey <sebastien.murgey@rte-france.com>
Signed-off-by: Sébastien Murgey <sebastien.murgey@rte-france.com>
Add isSecure(...) methods in RaoResult Signed-off-by: belthlemar <martin.belthle@rte-france.com>
Please check if the PR fulfills these requirements
Does this PR already have an issue describing the problem?
No
What kind of change does this PR introduce?
New feature
What is the new behavior (if this is a feature change)?
RaoResult objects now provides 3 new methods that allow client to assess the RAO result security status (i.e. positive margin on all Cnecs) , for multiple instants of optimization, and separately for each of the Cnec kind (Flow, Voltage and Angle).
These methods are :
This PR also add a method getLastInstant() to the CRAC API that allow to get the last instant based on order (biggest order one). This instant should correspond to the final optimisation instant.
Does this PR introduce a breaking change or deprecate an API?