docs(examples): the PyPSA file states a constraint once where PyPSA builds one row set, rather than a block per regime - #292
Merged
Conversation
…uilds one row set, rather than a block per regime (#257) Sixteen PyPSA names were stated by two to four blocks apiece, split by a regime the language had no way to name: the first snapshot a shift vacates, a cyclic store, an extendable build, a committed unit. `cases:` names the regime in the quantity instead, so each name is one block again. Claude-Session: https://claude.ai/code/session_01N8skeF7J54HymyTZayV3Ug (cherry picked from commit 88ce831) Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Note
The following content was generated by AI.
This is #257's diff, unchanged, onto
main.git cherry-pickof its merge commit5233263applies clean — six files,examples/and
docs/only, nosrc/— and the original authorship is preserved.#257 was stacked on
feat/cases-proved-apart, the branch carrying #168. #168 mergedto
mainon its own, which left that branch with no route there; merging #257 into itput the port on an orphaned base.
mainstill has zerocases:inexamples/pypsa.yaml, so no release can carry the port andv0.0.0-alpha.52cannot becut. Nothing is wrong with the change — only with where it landed.
The branch it merged into is now 2 ahead / 5 behind
main; it can be deleted once thislands.
What it is, unchanged from #257
Twelve PyPSA names — sixteen counted once per file — were stated by two to four blocks
apiece, split by a regime the language could not name inside a quantity. Each is now one
block.
pypsa.yamlpypsa_linearized_uc.yamlThe five that remain are the GlobalConstraint sense-splits, where the split is in the
operator rather than in a value — #239, closed
not planned, and untouched here.Verified on this base
pixi run cigreen onmain+ this commit: lint, 899 tests,mkdocs build --strict, 27 compiled documents. Tree clean after the run.The downstream ladder was run against this branch, which is the check docs(examples): the PyPSA file states a constraint once where PyPSA builds one row set, rather than a block per regime #257's own
symbolic argument could not make — lpspec's
differential/pypsa/parity.pybuilds eachrung through both engines and compares against live PyPSA 1.3.0. Rung 2 collapses
exactly as intended:
StorageUnit-energy_balanceandStore-energy_balancego fromtwo recorded structure deviations to none, at an unchanged objective and unchanged
row, column and dual counts.
Rungs 3 and up are not yet confirmed through that runner: lpspec's rung
projection — a presentation device that cuts the file down to what each rung builds
— does not understand
cases:yet, and its first cut mis-handles a region whosewhen:is negated. That is a consumer-side gap, filed as work on the lpspec side; itsays nothing about this diff, and
pixi run cihere does not depend on it.Not done here
second opinion about it. Review it there if it needs reviewing again.
feat/cases-proved-apartis left alone rather than force-moved ontomain.