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

#3209 add IEEE118 test case in Open Modelica examples #3225

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gaouizer
Copy link
Contributor

Checklist before requesting a review

use '[x]' to check the checkboxes, or submit the PR and then click the checkboxes

  • unit tests and non-regression tests were added (new model, new feature and bug fix)
  • main documentation was updated (update of input/output file, 3rd party, model, repository organization, solver)
  • example documentations were updated (new example in examples folder)
  • the corresponding milestone was added in the ticket and in this PR
  • if this PR modifies the parameters or inputs/outputs of a model/solver: the corresponding xsl was added in util/xsl and platform DB were updated
  • if this PR modifies a dictionary: the corresponding french dictionary was updated

@gaouizer gaouizer added this to the v1.7.0 milestone May 17, 2024
@gaouizer gaouizer self-assigned this May 17, 2024
@joyelfeghali joyelfeghali linked an issue May 17, 2024 that may be closed by this pull request
@@ -0,0 +1,22 @@
# Copyright (c) 2023, RTE (http://www.rte-france.com)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2024 for all files

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can remove this file

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You could add references results


//Annotations and calculation parameters
annotation(preferredView = "text",
experiment(StartTime = 0, StopTime = 500, Tolerance = 1e-06, Interval = 10),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you remove the experiment line here?

Dynawo.Electrical.Machines.SignalN.GeneratorPVTfo B116_G1(KGover = 0.0, PGen0Pu = 0.0, PMaxPu = 100.0, PMinPu = -100.0, PNom = 100.0, PRef0Pu = -0.0, QGen0Pu = 0.5132, QMaxPu = 10.0, QMinPu = -10.0, QNomAlt = 1000.0, QStator0Pu = 0.0514, SNom = 10050.0, U0Pu = 1.0, UStator0Pu = 1.0006, UStatorRef0Pu = 1.0056, XTfoPu = 0.1228, SwitchOffSignal10 = false, SwitchOffSignal20 = false, SwitchOffSignal30 = false, limUQDown0 = false, limUQUp0 = false, qStatus0 = Dynawo.Electrical.Machines.SignalN.GeneratorPV.QStatus.Standard, i0Pu = Complex(0.0, 0.5132), iStator0Pu = Complex(0.0, -0.0051), sStator0Pu = Complex(0.0, 0.0051), u0Pu = Complex(1.0, 0.0), uStator0Pu = Complex(1.0006, 0.0));

//Loads
Dynawo.Electrical.Loads.LoadAlphaBetaRestorative B1_L1(Alpha = 1.5, Beta = 2.5, UMaxPu = 1.15, UMinPu = 0.85, i0Pu = Complex(0.4536, -0.3258), s0Pu = Complex(0.476, 0.2407), tFilter = 10.0, u0Pu = Complex(0.9437, -0.1471));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since all values of Alpha, Beta, UMinPu, UMaxPu and filter are the same, you could define parameters like what's done in IEEE14 :
parameter Real Alpha = 1.5 "Active load sensitivity to voltage";
parameter Real Beta = 2.5 "Reactive load sensitivity to voltage";
parameter Types.VoltageModulePu UMaxPu = 1.05 "Maximum value of the voltage amplitude at terminal in pu (base UNom) that ensures the P/Q restoration";
parameter Types.VoltageModulePu UMinPu = 0.95 "Minimum value of the voltage amplitude at terminal in pu (base UNom) that ensures the P/Q restoration";
parameter Types.Time tFilter = 10;

* of simulation tools for power systems.
*/
model IEEE118Base "Base class for IEEE 118-bus system benchmark formed with 118 buses, 54 generators, 14 shunts, 9 transformers , 177 lines and 99 loads."
final parameter Modelica.SIunits.Impedance ZBASE138_0 = 138.0 ^ 2 / Dynawo.Electrical.SystemBase.SnRef;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is only one Zbase ?

B110_SH1.switchOffSignal1.value = false;
B110_SH1.switchOffSignal2.value = false;

//Loads optional variables
Copy link
Contributor

@joyelfeghali joyelfeghali Jun 3, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Loads inputs

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

Successfully merging this pull request may close these issues.

Add the IEEE118 test case in the Modelica examples
2 participants