Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Faults and fault clearing events incorrectly modify shunts #2

Open
susantoj opened this issue Mar 21, 2015 · 3 comments
Open

Faults and fault clearing events incorrectly modify shunts #2

susantoj opened this issue Mar 21, 2015 · 3 comments
Assignees
Labels

Comments

@susantoj
Copy link
Owner

Faults are modelled as high shunt admittances / conductances and use the shunt columns (Gs, Bs) in the PYPOWER case file. This is okay if there are no actual shunts on the bus, but where a shunt exists, then the fault and fault clearing events overwrite the existing shunt values.

Need to fix this.

@susantoj susantoj added the bug label Mar 21, 2015
@susantoj susantoj self-assigned this Mar 21, 2015
@rwl
Copy link
Collaborator

rwl commented Jun 27, 2015

One solution to this might be to add separate Load and Shunt classes to the case structure proposed in #9.

As with the Generator class this would allow more than one device at the same bus. They would also have a status attribute to allow temporary omission from calculations. In the future the Load class could be extended with attributes to support voltage dependency.

Implementing this would require a reasonable amount of effort as Pd, Qd, Gs, and Bs are used in quite a few different places.

@susantoj
Copy link
Owner Author

Having separate Load and Shunt classes would be the ideal structure for the case file to give it more flexibility (e.g. the Load class can also have flags for motor / non-motor loads).

But like you say, there will be some work required to make sure the new classes can be integrated into the existing code, particularly for Ybus matrix construction. I think we need to settle on a new, more comprehensive case definition before starting work on this.

@rwl
Copy link
Collaborator

rwl commented Jun 29, 2015

Integrating the changes with the existing code should be made much simpler with these (as yet untested) functions:

rwl@51c48a8

They return complex vectors of length bus.n and should account for there potentially being more than one device per bus. The commit includes the necessary changes to makeYbus and they are quite minimal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants