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 power flow initialisation & updating for the terminals #223

Closed
gnakti opened this issue Jun 19, 2023 · 0 comments · Fixed by #238
Closed

Fix power flow initialisation & updating for the terminals #223

gnakti opened this issue Jun 19, 2023 · 0 comments · Fixed by #238
Assignees
Labels
bug Something isn't working

Comments

@gnakti
Copy link
Contributor

gnakti commented Jun 19, 2023

  • see PR Fix power flow initialisation of synchronous generators #238
  • The terminals of the components are not correctly initialized with initial power and updated with power flow solutions. This might lead to an initialization error in the dynamic simulation (if the terminal power is used in initializeFromNodesAndTerminals e.g. terminal(0)->singlePower())
@gnakti gnakti added the bug Something isn't working label Jun 19, 2023
@martinmoraga martinmoraga self-assigned this Jul 26, 2023
@martinmoraga martinmoraga linked a pull request Jul 27, 2023 that will close this issue
dinkelbachjan added a commit that referenced this issue Nov 30, 2023
In powerflow simulations, the power of synchronous generators is not
being always correctly calculated, since the power of the loads
connected to nodes is not considered in the calculation. This PR fixes
the calculation of the synchronous generators' power in powerflow
simulations and adapts the funcion `initWithPowerFlow` to initialialize
the terminal power of the synchronous generators after the powerflow
calculation.

Closes #223
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants