Sienna-Platform / Sienna-Platform/PowerOperationsModels.jl
Generalize (aux_var, device_type, results-column) correspondences instead of hard-coding them
Open
@luke-kiernan is already working on this.
Since Jul 22, 2026.
wontfix
- Dominant language
- Julia
- Stars
- 2
- Forks
- 1
- Avg merge
- 1d 16h
- Merged PRs (30d)
- 26
Description
HVDC power-flow aux variables are wired through a hand-maintained tuple-of-tuples, _HVDC_AUX_SPECS in src/network_models/power_flow_evaluation.jl, each row being (aux_var_type, component_type, results_table, name_column, value_column, is_natural_units_power). That one const drives registration (hvdc_aux_vars), the fill loop (_fill_hvdc_aux_var!), and unit conversion. It works and is already a consolidation, but the (optimization key, device type, results field) shape recurs across the codebase.
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.
Assessment
This issue has not been assessed yet.