Sienna-Platform / Sienna-Platform/PowerFlowFileParser.jl
Zero base power
Nobody has claimed this yet.
- Dominant language
- Julia
- Stars
- 1
- Forks
- 2
- Avg merge
- 22h 29m
- Merged PRs (30d)
- 10
Description
Looks like the behavior on this issue has regressed.
We should disallow 0.0 as a base power for components. Otherwise, you can get thoroughly confusing things like PSB issue https://github.com/NREL-Sienna/PowerSystemCaseBuilder.jl/issues/174: get_reactive_power(comp) yields 0.0, but calling set_reactive_power!(comp, 0.0) then changes the reactive power to NaN.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating validation for component base power and trace the get_reactive_power and set_reactive_power! entry points mentioned in the issue. Reproduce the zero-base-power case and confirm that 0.0 is rejected rather than allowing set_reactive_power! to produce NaN; the linked regression issue provides additional context.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- julia
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100