Skip to content

Iterative Linear State Estimation: power sensor accuracy #951

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

Conversation

mgovers
Copy link
Member

@mgovers mgovers commented Apr 10, 2025

The Iterative Linear State Estimation algorithm assumes that linearized rescaling power to current keeps the variances the same. This assumption is true when node voltages are close to 1 p.u.. However, when voltages may vary significantly, this assumption becomes increasingly bad, and may lead to incorrect weighing of different measurements, and may even lead to incorrect answers.

This becomes especially important when looking at grids with both power and current sensors.

TBD (do-not-merge label added):

  • the new implementation causes the prefactorization to happen on a per-iteration level, instead of on a global level.
    • What is the computational impact of that? (benchmark)
    • Is this change desirable?
  • the new implementation may lead to slightly different answers from the original implementation
    • What is the user impact of that?

mgovers added 3 commits April 9, 2025 14:57
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers mgovers added improvement Improvement on internal implementation do-not-merge This should not be merged labels Apr 10, 2025
@mgovers mgovers self-assigned this Apr 10, 2025
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers mgovers force-pushed the feature/ilse-power-sensor-accuracy branch from 2fd4c50 to f621ab2 Compare April 10, 2025 15:18
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers mgovers force-pushed the feature/ilse-power-sensor-accuracy branch from 92bff8e to 83b4c48 Compare April 11, 2025 07:21
@mgovers mgovers changed the title Feature/ilse power sensor accuracy Iterative Linear State Estimation: power sensor accuracy Apr 14, 2025
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers mgovers removed the do-not-merge This should not be merged label Apr 15, 2025
@mgovers mgovers changed the base branch from main to feature/add-scaling-to-statistics April 15, 2025 09:39
…wer-sensor-accuracy

Signed-off-by: Martijn Govers <martijn.govers@alliander.com>
@mgovers mgovers added the do-not-merge This should not be merged label Apr 15, 2025
@figueroa1395
Copy link
Contributor

This one can now be closed after offline conclusions, right? And then add the documentation needed to the issue #547, correct?

@mgovers mgovers force-pushed the feature/add-scaling-to-statistics branch 3 times, most recently from 850e013 to a206d17 Compare April 15, 2025 12:00
@TonyXiang8787
Copy link
Member

@mgovers based on the decision. We should close this PR, right?

@mgovers
Copy link
Member Author

mgovers commented Apr 15, 2025

This one can now be closed after offline conclusions, right? And then add the documentation needed to the issue #547, correct?

@mgovers based on the decision. We should close this PR, right?

Indeed. To summarize the discussions:

  • The variances are indeed not one-to-one scaled.
  • The power sensors used in ILSE are an approximation.
    • Uncertainties/sigmas of the measurements are typically best-effort values.
    • Linearization of the measurements and error margins are merely an approximation in itself.
    • Interpretation of power measurements as current measurements is also an assumption.
  • It is probably not a huge issue that the power sensor variances are not correctly handled within the domain in which linearization is valid and in which we are operating.
  • The inner-loop factorization of the matrix would come at a significant computational cost.

Based on these arguments, we decided to go with the following:

  • Present ILSE as an approximate method and properly document the assumptions that are made.

The following options were also considered but rejected in favor of the chosen approach.

* [ ] Do the change and present it as an improvement to the ILSE
* [ ] Do the change and present it as a bugfix
* [ ] Add the change as a separate calculation method to the original ILSE

In conclusion, this PR will be closed and superseded by a follow-up documentation PR.

@mgovers mgovers closed this Apr 15, 2025
@mgovers mgovers deleted the feature/ilse-power-sensor-accuracy branch April 15, 2025 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge This should not be merged improvement Improvement on internal implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants