-
Notifications
You must be signed in to change notification settings - Fork 22
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
25 ibg #3012
base: master
Are you sure you want to change the base?
25 ibg #3012
Conversation
eee050a
to
2f493e8
Compare
52538cc
to
1601da8
Compare
dynawo/sources/Common/Dictionaries/DYNTimeline_oppositeEvents.dic
Outdated
Show resolved
Hide resolved
dynawo/sources/Models/Modelica/Dynawo/Electrical/Controls/PLL/PLLFreeze.mo
Outdated
Show resolved
Hide resolved
dynawo/sources/Models/Modelica/Dynawo/Electrical/InverterBasedGeneration/AggregatedIBG.mo
Outdated
Show resolved
Hide resolved
dynawo/sources/Models/Modelica/Dynawo/Electrical/InverterBasedGeneration/GenericIBG.mo
Outdated
Show resolved
Hide resolved
dynawo/sources/Models/Modelica/Dynawo/Electrical/InverterBasedGeneration/GenericIBG.mo
Outdated
Show resolved
Hide resolved
dynawo/sources/Models/Modelica/Dynawo/Electrical/InverterBasedGeneration/GenericIBG.mo
Outdated
Show resolved
Hide resolved
dynawo/sources/Models/Modelica/Dynawo/Electrical/InverterBasedGeneration/GenericIBG.mo
Outdated
Show resolved
Hide resolved
Hello, I am currently implementing the WECC der_a model and I plan to add it to this PR since it has a few common building blocks. I am quite busy at the moment but should normally be able to update the PR and account for the comments early February. |
5c59fef
to
69807ea
Compare
Added the der_a model and updated PR following comments |
In the current implementation, the solver (either IDA or SIM) will sometimes fail with The following assertion has been violated %sat time %f This is strange because the current limitation model is the same as in for WECC PV plants (WECC.BaseControls.CurrentLimitsCalculationPV). This occurs when iP increases leading to iQmax and iQmin=-iQmax "cross" at 0 due to extrapolation. @rosiereflo : I haven't tested this extensively, but in the attached test case, removing the noEvent in the equation below solves the issue. iqMaxPu = noEvent(if (abs(IMaxPu) > abs(ipLimPu)) then sqrt(IMaxPu ^ 2 - ipLimPu ^ 2) else 0); |
…o current implementation of PLL model Signed-off-by: FredericSabot <[email protected]>
Signed-off-by: FredericSabot <[email protected]>
Signed-off-by: FredericSabot <[email protected]>
a2c2164
to
f434d74
Compare
Signed-off-by: FredericSabot <[email protected]>
Signed-off-by: FredericSabot <[email protected]>
closes #3011
Checklist before requesting a review
use
'[x]'
to check the checkboxes, or submit the PR and then click the checkboxes