Skip to content
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

Fix second preventive scenario #874

Merged
merged 5 commits into from
Jan 25, 2024
Merged

Fix second preventive scenario #874

merged 5 commits into from
Jan 25, 2024

Conversation

pet-mit
Copy link
Collaborator

@pet-mit pet-mit commented Jan 25, 2024

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)

What kind of change does this PR introduce?
Bug fix

What is the current behavior?
The fixes in #866 reset the network used in 2nd preventive RAO to initial state before last load flow computation. However, if the 2nd preventive is not global, this makes it lose PST taps that were copied from the 1st preventive result.

What is the new behavior (if this is a feature change)?
Now the network is reset to the situation before 2nd PRAO, which contains 1st PRAO PST taps.

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

  • Yes
  • No

@pet-mit pet-mit added bug Something isn't working PR: waiting-for-review This PR is waiting to be reviewed labels Jan 25, 2024
@pet-mit pet-mit requested a review from phiedw January 25, 2024 08:52
@pet-mit pet-mit merged commit 2b14ec7 into main Jan 25, 2024
6 checks passed
@pet-mit pet-mit deleted the create-initial-scenario branch January 25, 2024 12:07
MartinBelthle pushed a commit that referenced this pull request May 28, 2024
Fix second preventive scenario

Signed-off-by: belthlemar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working PR: waiting-for-review This PR is waiting to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants