Skip to content

adding winding number for regulators#76

Open
zzink-nrel wants to merge 1 commit into
mainfrom
zz/tapped_winding
Open

adding winding number for regulators#76
zzink-nrel wants to merge 1 commit into
mainfrom
zz/tapped_winding

Conversation

@zzink-nrel

@zzink-nrel zzink-nrel commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

An issue arises in writing OpenDSS voltage regulators where the controlled winding is not specified and OpenDSS defaults to winding=1 which is not the typical expected behavior of controlling the low side.

This was not discovered in tests because the tests disable controlled assets. By inspection, the regulators in IEEE13 would fail to be reproduced.

I have added it this way (hard-coded) because GDM has no field that captures this information. If supported by others, I could add thie to GDM:

class RegulatorController(Component):

    ...

    tapped_winding: Annotated[
        Optional[int],
        Field(
            2,
            ge=0,
            description="The index of the winding on the transformer that this controller is controlling.",
        ),
    ]

and read it in properly:

def map_tapped_winding(self):
    self.opendss_dict["Winding"] = self.model.tapped_winding

@zzink-nrel zzink-nrel requested a review from tarekelgindy June 11, 2026 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant