-
Notifications
You must be signed in to change notification settings - Fork 2
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
[FEATURE] Ability to replace a branch measurement with an injection measurement (And subsequent changes to the grid) #26
Comments
Thank you for the feature idea. How would you propose collapsing the subgrid loads, summing all loads in the subgrid, grouped by types? On implementation, we have a function to get all downstream nodes from the cut-off already: power-grid-model-ds/src/power_grid_model_ds/_core/model/grids/base.py Lines 332 to 351 in 00a7d2d
If you would add this I would prefer setting up a helper function section instead of adding it as a function to the grid itself. To keep it focussed on basic grid interaction and modification |
In state estimation, we do not need to sum up loads. We are essentially "ignoring" the The strategy about this would be following (Easier than mentioned in the description):
This doesnt address a meshed grid though, so more thought is needed here. Also, helper function would do! |
Thanks for the clarification. So my strategy would be to
How would you like to procede? Is this something you can draft a PR for? Or do you want to discuss a design proposal first? |
While running State Estimation, many times its required to collapse a section of the grid to a smaller grids with less components because we might not have complete observability of the subsection from measurements perspective. This is useful when we have a detailed form of grid data but we only have requirement to run estimation for upstream nodes. PGM DS can thus provide a safe way to implement this.
@IrenaDk / @francisca-mestre, this might be useful for you.
For eg:
can be converted to
There would be more features required to implement this. The end goal is described here.
The text was updated successfully, but these errors were encountered: