Conversation
Regenerated all seven packages from SiennaSchemas main, which adds the remote_regulated_bus_id and voltage_setpoint fields, the transformer regulated_bus_id/regulated_bus_side/load_drop_compensation block, the LCC commutating-bus and tap-transformer references, and the VoltageDroopControl, ReactivePowerSharing and VoltageControlAssociation types, and removes the per-device rmpct fields. The regeneration also catches up with the schema commits since the last one (units sentences on the curve types, the dynamics component schemas, the removal of TopologyMapping). The hand-written SystemDocument container gains voltage_control_associations, optional and defaulting to empty like trading_hub_associations, with a (control, entity, terminal) duplicate guard and reference validation. The three new Operations types join the registered document component set. Test fixtures follow the schemas: RoundRotorMachine no longer carries the derived gamma fields, the portfolio fixture uses RetirementPotential, and DEGOV1 is recognized as a governor model distinct from DEGOV rather than a generated alias.
This was referenced Sep 28, 2026
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.
Remote and shared voltage control
Regenerates all seven packages from the SiennaSchemas
mb/remote-controlbranch.What changes
remote_regulated_bus_idandvoltage_setpointon voltage-regulating devices,regulated_bus_id/regulated_bus_side/load_drop_compensation_r/_xon transformer circuits, and the LCC commutating-bus and tap-transformer references.VoltageDroopControl,ReactivePowerSharing,VoltageControlAssociation, and theTransformerRegulatedBusSideandVoltageControlTerminalenums, each with anUNDEFINEDmember.rmpctfields are removed.SystemDocumentcontainer gainsvoltage_control_associations, optional and empty by default liketrading_hub_associations, with a duplicate guard on (control, entity, terminal) and reference validation.Admittancequantity kind, units sentences on the curve types, the dynamics component schemas, numeric governor defaults and the removal ofTopologyMapping.Before merge
mainin generated files since Validate each value once instead of once per nesting level #26. It needs a rebase and a fresh regeneration, not a hand merge.mainonce Remote and shared voltage control SiennaSchemas#67 merges, so.schema-versionrecords a real schema revision.Verification
473 tests pass (2026-09-24).
Stack
Merge in this order.
The Julia branches pin each other's
mb/remote-controlbranches in[sources], and each pin moves back tomainorpsy6as its dependency merges.Depends on: Sienna-Platform/SiennaSchemas#67.
🤖 Generated with Claude Code