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 corner cases of zero impedance subgraph #765

Open
flo-dup opened this issue Mar 31, 2023 · 0 comments
Open

Fix corner cases of zero impedance subgraph #765

flo-dup opened this issue Mar 31, 2023 · 0 comments

Comments

@flo-dup
Copy link
Contributor

flo-dup commented Mar 31, 2023

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
Some corner cases not covered:

  1. VoltageControl::findVoltageControlsSortedByPriority only checks if the controlled bus is disabled or all the controller buses are disabled, and does not check LfBus::isGeneratorVoltageControlEnabled(). All the controller buses can be enabled but with all the connected generators in contingency.
  2. For a newly disabled branch which is a voltage controller, the corresponding voltage control can be selected as the main one if its target voltage is the highest for its type. If it is the main one, we dismiss it later, in VoltageLevel::findVoltageControlsSortedByPriority: the control is considered disabled, and then we don't consider any voltage control of that type.

What is the motivation / use case for changing the behavior?
To support corner cases, and cover them in unit tests. Or clarify why it could not happen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant