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

#3253 Add the 2015 version of the IEC WPP models #3274

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

louisg-rte
Copy link
Collaborator

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

@rosiereflo
Copy link
Contributor

Thanks for adding the python script!

@rosiereflo rosiereflo added this to the v1.7.0 milestone Jun 12, 2024
@rosiereflo rosiereflo linked an issue Jun 12, 2024 that may be closed by this pull request
@louisg-rte louisg-rte force-pushed the 3253_IEC_61400_27_WPP_models branch from fff987c to 505e3b6 Compare June 18, 2024 08:57
@louisg-rte louisg-rte force-pushed the 3253_IEC_61400_27_WPP_models branch 2 times, most recently from 9648278 to f9b3ea4 Compare June 27, 2024 12:33
GARBAY Louis added 3 commits June 28, 2024 18:59
Signed-off-by: GARBAY Louis <garbaylou@gm0winl417.bureau.si.interne>
Signed-off-by: GARBAY Louis <garbaylou@gm0winl417.bureau.si.interne>
Signed-off-by: GARBAY Louis <garbaylou@gm0winl417.bureau.si.interne>
@louisg-rte louisg-rte force-pushed the 3253_IEC_61400_27_WPP_models branch from f9b3ea4 to a14eee4 Compare June 28, 2024 16:59
Placement(visible = true, transformation(origin = {110, -90}, extent = {{-10, -10}, {10, 10}}, rotation = 0)));
Modelica.Blocks.MathBoolean.Or or1(nu = 2) annotation(
Placement(visible = true, transformation(origin = {110, -40}, extent = {{10, -10}, {-10, 10}}, rotation = 0)));
Dynawo.NonElectrical.Blocks.Continuous.TransferFunction transferFunction(a = {tpfv, 1}, b = {tpft, 1}, x_start = {-P0Pu * SystemBase.SnRef / SNom}, y_start = -P0Pu * SystemBase.SnRef / SNom) annotation(
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 use the transfer function block of the Modelica library?

Placement(visible = true, transformation(origin = {-290, -40}, extent = {{-10, -10}, {10, 10}}, rotation = 0)));
Dynawo.NonElectrical.Blocks.Continuous.AbsLimRateLimFeedthroughFreezeLimDetection absLimRateLimFeedthroughFreezeLimDetection1(DyMax = DXRefMaxPu, DyMin = DXRefMinPu, U0 = XWT0Pu, Y0 = XWT0Pu, YMax = XRefMaxPu, YMin = XRefMinPu, tS = tS) annotation(
Placement(visible = true, transformation(origin = {326, 100}, extent = {{-10, -10}, {10, 10}}, rotation = 0)));
Dynawo.NonElectrical.Blocks.Continuous.TransferFunction transferFunction(a = {txfv, 1}, b = {txft, 1}, x_start = {XWT0Pu}, y_start = XWT0Pu) annotation(
Copy link
Contributor

Choose a reason for hiding this comment

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

same here

<curvesInput xmlns="http://www.rte-france.com/dynawo">
<curve model="WPP4A" variable="WPP_wPPControl_measurements_PGenNomPu"/>
<curve model="WPP4A" variable="WPP_wPPControl_measurements_QGenNomPu"/>
<curve model="WPP4A" variable="WPP_wPPControl_measurements_UPu"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

replace these three variables with the ones of 2015. For all the 2015 test cases

<curve model="WPP4A" variable="WPP_terminal_V_im"/>
<curve model="WPP4A" variable="WPP_terminal_i_re"/>
<curve model="WPP4A" variable="WPP_terminal_i_im"/>
<curve model="WPP4A" variable="WPP_wT4ACurrentSource_gridProtection_fOCB"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

we could add also the timeline for this test case

<curve model="WPP4B" variable="WPP_terminal_V_im"/>
<curve model="WPP4B" variable="WPP_terminal_i_re"/>
<curve model="WPP4B" variable="WPP_terminal_i_im"/>
<curve model="WPP4B" variable="WPP_wT4BCurrentSource_gridProtection_fOCB"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

we could also add timeline for this test case

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.

Add the 2015 version of the IEC-61400-27 WPP models.
3 participants