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

"get_reactive_power_limits not implemented for RenewableNonDispatch" #39

Closed
GabrielKS opened this issue Jul 15, 2024 · 4 comments · Fixed by #79
Closed

"get_reactive_power_limits not implemented for RenewableNonDispatch" #39

GabrielKS opened this issue Jul 15, 2024 · 4 comments · Fixed by #79

Comments

@GabrielKS
Copy link
Contributor

Certain power flow runs fail with this message:
Screenshot 2024-07-12 at 1 48 26 PM

It looks like there is a possible solution here from @HaleyRoss.

@GabrielKS
Copy link
Contributor Author

For reference, a quick workaround for this if you just need something to run is:

PowerSystems.get_reactive_power_limits(::RenewableNonDispatch) = (min = 0.0, max = 0.0)

@GabrielKS
Copy link
Contributor Author

We have similars issues with get_active_power_limits, with similar workarounds that look like this:

PowerSystems.get_active_power_limits(::RenewableDispatch) = (min = 0.0, max = 0.0)

A principled solution is also needed for get_active_power_limits.

@GabrielKS
Copy link
Contributor Author

GabrielKS commented Jan 15, 2025

Plan for get_reactive_power_limits(gen::RenewableNonDispatch): since the reactive power is fixed, this should return (min = get_reactive_power(gen), max = get_reactive_power(gen)).

@GabrielKS
Copy link
Contributor Author

Plan in consultation with @jd-lara:

  • Do not implement PSY.get_reactive_power_limits for RenewableNonDispatch, but insert logic into PowerFlows to behave as if the min and max were both the current reactive power setpoint
  • Similarly, do not implement PSY.get_active_power_limits for the types on which it is missing, but act in PowerFlows as if:
    • for Source: infinite limits
    • for RenewableNonDispatch: min = max = setpoint
    • for RenewableDispatch: min = 0, max = rating

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.

1 participant