Sienna-Platform / Sienna-Platform/PowerOperationsModels.jl
Network reductions and constraint duals
Nobody has claimed this yet.
- Dominant language
- Julia
- Stars
- 2
- Forks
- 1
- Avg merge
- 1d 16h
- Merged PRs (30d)
- 26
Description
Quoting from IOM's add_constraint_dual.jl in an about-to-be-merged PR:
Reuse the existing constraint container's row axis so the dual axis
matches the constraint exactly. Network reductions (radial /
degree-two) drop branches that pass the device-model filter, so the
constraint axis is a strict subset of IS.get_name.(devices). Sizing
the dual from the device list would leave the dual broadcast in
process_duals incompatible with the constraint matrix.
Add a dedicated test case for that.
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 reading the referenced add_constraint_dual.jl change in IOM and tracing process_duals alongside this repository's network-reduction code. Add a dedicated test covering a constraint axis that is a strict subset of the device list, and verify that dual processing remains compatible with the constraint matrix.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- julia
- Domain
- testing-qa
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 62/100