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

Faulty parameter propagation in IntersectivePWM #4089

Open
christiankral opened this issue Mar 12, 2023 · 0 comments · May be fixed by #4090
Open

Faulty parameter propagation in IntersectivePWM #4089

christiankral opened this issue Mar 12, 2023 · 0 comments · May be fixed by #4090
Assignees
Labels
bug Critical/severe issue L: Electrical.PowerConverters Issue addresses Modelica.Electrical.PowerConverters
Milestone

Comments

@christiankral
Copy link
Contributor

In the model Modelica.Electrical.PowerConverters.DCAC.Control.IntersectivePWMIntersectivePWM there are issue with the propagation of three parameters:

1 Array size of trapezoid

Modelica.Blocks.Sources.Trapezoid trapezoid[3](...);

should be

Modelica.Blocks.Sources.Trapezoid trapezoid[m](...);

2 Start time porpagated to trapezoid

  Modelica.Blocks.Sources.Trapezoid trapezoid[3](
    each final amplitude=uMax,
    each final width=0,
    each final nperiod=-1,
    each final offset=-uMax/2,
    each final rising=0.5/f,
    each final falling=0.5/f,
    each final period=1/f,
    final startTime={startTime - 1.25 + (if refType == ReferenceType.Triangle1
         then 0 else k)/m for k in 0:m - 1}/f)
         if (refType==ReferenceType.Triangle1 or refType==ReferenceType.Triangle3);

The units of the expression final startTime=startTime - 1.25 + (if refType == ReferenceType.Triangle1 then 0 else k)/m for k in 0:m - 1}/f do not match as startTime is added 1.25 and other real expressions.

3 Start time porpagated to sawTooth

Similar issue as 2. The propagation reads final startTime={startTime - 1.5 + (if refType == ReferenceType.Sawtooth1 then 0 else k)/m for k in 0:m - 1}/f


I understand the different phase start times associated with Triangle1 and Triangle3 but I can only suspect that the term 1.5 shall ensure that

  • all the reference signals are fully up at time >= startTime
  • the first reference signal start with half the maximum value

So my understanding for a three phase saw tooth PWM is:

sawTooth3

@christiankral christiankral added the bug Critical/severe issue label Mar 12, 2023
@christiankral christiankral added this to the maintenance milestone Mar 12, 2023
@christiankral christiankral added the L: Electrical.PowerConverters Issue addresses Modelica.Electrical.PowerConverters label Mar 12, 2023
@beutlich beutlich modified the milestones: maintenance, MSL4.1.0 Apr 17, 2023
@TManikantan TManikantan linked a pull request May 5, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Critical/severe issue L: Electrical.PowerConverters Issue addresses Modelica.Electrical.PowerConverters
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants