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.

Sign up for GitHub

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

Jump to bottom

mvasc1 default #176

Closed
sanderclaeys opened this issue Sep 13, 2019 · 0 comments · Fixed by #178
Closed

mvasc1 default #176

sanderclaeys opened this issue Sep 13, 2019 · 0 comments · Fixed by #178

Comments

@sanderclaeys
Copy link
Contributor

When only MVAsc3 is specified and not MVAsc1 (such as IEEE34), the numerical results differ between OpenDSS and PMD. This leads me to believe that the default value for MVAsc1 is not mapped correctly.

This is not an important element of IEEE34, as MVAsc3 is set to a large value to model a very stiff source; I will assume the same for MVAsc1. This does not have to be fixed prior to the PSCC deadline.

I have been able to verify all feeders to a very high accuracy (1E-7).
pseudocubic added a commit that referenced this issue Sep 23, 2019
@pseudocubic
FIX: setting source values
030039c 
Fixes the creation of vsource from circuit to use correct values for mvasc3,mvasc1 and isc3,isc1  if one of the pair is not specified.

Default values in documentation are not consistent with software for ISC(1|3)

Updates unit tests and changelog

Closes #176
@pseudocubic pseudocubic mentioned this issue Sep 23, 2019
Merged
pseudocubic added a commit that referenced this issue Sep 23, 2019
@pseudocubic
FIX: setting source values (#178)
61055fc 
Fixes the creation of vsource from circuit to use correct values for mvasc3,mvasc1 and isc3,isc1  if one of the pair is not specified.

Default values in documentation are not consistent with software for ISC(1|3)

Updates unit tests and changelog

Closes #176
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

FIX: setting source values lanl-ansi/PowerModelsDistribution.jl
1 participant
@sanderclaeys