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

Modify PowerFlowData to support multi-period AC power flow data #77

Closed
GabrielKS opened this issue Jan 13, 2025 · 3 comments · Fixed by #78
Closed

Modify PowerFlowData to support multi-period AC power flow data #77

GabrielKS opened this issue Jan 13, 2025 · 3 comments · Fixed by #78

Comments

@GabrielKS
Copy link
Contributor

I believe these are the only fields that need to be modified:

  • bus_reactivepower_bounds::Vector{Vector{Float64}} -> bus_reactivepower_bounds::Matrix{Vector{Float64}}
  • branch_type::Vector{DataType} -> branch_type::Matrix{DataType}

but code changes to account for these modifications are also needed.

@GabrielKS
Copy link
Contributor Author

GabrielKS commented Jan 13, 2025

One more: we need to add a field branch_reactivepower_flow_values::Matrix{Float64} (and modify existing branch_flow_values::Matrix{Float64} -> branch_activepower_flow_values::Matrix{Float64}).

@GabrielKS
Copy link
Contributor Author

We also think we might have to split each of the flow fields into to->from and from->to to account for losses (as the DataFrame outputs already do).

@GabrielKS
Copy link
Contributor Author

Closing as completed by #78

@GabrielKS GabrielKS linked a pull request Jan 16, 2025 that will close this issue
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