Skip to content

Issue/1147#56

Merged
poorva1209 merged 7 commits intodevelopfrom
issue/1147
Nov 4, 2019
Merged

Issue/1147#56
poorva1209 merged 7 commits intodevelopfrom
issue/1147

Conversation

@temcdrm
Copy link
Copy Markdown

@temcdrm temcdrm commented Oct 30, 2019

This writes SWING_PQ nodes wherever a SynchronousMachine or Storage Inverter is connected to a non-triplex node, so it will support an island. The ieee8500_3subs, acep_psil and ieee13 models fall into this category. They don't actually support an island until switches and generators are dispatched to create the island. All models in the platform were tested from OpenDSS through CIM to GridLAB-D and OpenDSS power flow comparison. The base results don't change from before.

Take note of the three new lines in MakeGlmTestScript.py. Those lines have to be written by the platform as well; not sure where that code is. If a model doesn't use SWING_PQ nodes there will be no effect. If a model does use SWING_PQ nodes, it won't run without these three new lines.

In addition, there are six generators in ieee8500_3subs that now appear in the CIM XML, but with p=q=0 for no output. These can support islands if they are turned on. Before, they were not available in the CIM XML because they had been disabled in OpenDSS.

BTW, this pull request has two sample queries that provide nominal values for the sensor service.


This change is Reviewable

… fault_check object is only written once, for external fault scripting and/or SWING_PQ node islanding
@temcdrm
Copy link
Copy Markdown
Author

temcdrm commented Oct 31, 2019

@poorva1209 look at the October 31 commit changes to CIMImporter.java. The platform is probably calling start, generateGlmFile or WriteGLMFile from some location. Wherever that occurs, there is a new argument bHaveEventGen that should be true if you've already written a fault_check and eventgen object for external fault scripting, and false otherwise.

@poorva1209 poorva1209 merged commit 89d27f3 into develop Nov 4, 2019
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.

2 participants