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

Bug in the weeds of _reactive_power_redistribution_pv #44

Closed
GabrielKS opened this issue Jul 25, 2024 · 2 comments · Fixed by #45
Closed

Bug in the weeds of _reactive_power_redistribution_pv #44

GabrielKS opened this issue Jul 25, 2024 · 2 comments · Fixed by #45
Assignees

Comments

@GabrielKS
Copy link
Contributor

The helper function _reactive_power_redistribution_pv:

function _reactive_power_redistribution_pv(sys::PSY.System, Q_gen::Float64, bus::PSY.Bus)

takes a system, a reactive power, and a bus; and distributes that reactive power among the devices that contribute to the bus. We would therefore expect that after this function is run, the sum of the reactive power among the relevant devices is equal to the Q_gen provided to the function. However:

Screenshot 2024-07-22 at 3 44 17 PM

it isn't.

@GabrielKS
Copy link
Contributor Author

GabrielKS commented Jul 25, 2024

@jd-lara could you verify that this is in fact a bug? It should definitely be possible to distribute that amount of reactive power because that amount of reactive power is already distributed.

@GabrielKS
Copy link
Contributor Author

I have a test here that would fail if I did not manually exclude reactive power from the comparison. The fact that it works for active power (and voltage magnitude and voltage angle) suggests that this is indeed a bug.

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

Successfully merging a pull request may close this issue.

2 participants