-
Notifications
You must be signed in to change notification settings - Fork 168
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
Implement choice between sawtooth and triangle reference signal in PowerConverters/DCDC/Control/SignalPWM #3679
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
Omitting the full path to the option could lead to problems in some tools.
I'd prefer to keep it as it is.
Am Fr., 8. Jan. 2021 um 09:34 Uhr schrieb tobolar <[email protected]
…:
***@***.**** commented on this pull request.
------------------------------
In Modelica/Electrical/PowerConverters/DCDC/Control/SignalPWM.mo
<#3679 (comment)>
:
> parameter Boolean useConstantDutyCycle=true
"Enables constant duty cycle";
parameter Real constantDutyCycle=0 "Constant duty cycle"
annotation (Dialog(enable=useConstantDutyCycle));
parameter SI.Frequency f=1000 "Switching frequency";
+ parameter SingleReferenceType refType=Modelica.Electrical.PowerConverters.Types.SingleReferenceType.Sawtooth
Possibly ... refType=SingleReferenceType.Sawtooth ?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#3679 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABUUUMNY7AKOWD7CISDAY2DSY27RNANCNFSM4UMHRBQQ>
.
|
…werConverters/DCDC/Control/SignalPWM
Please be aware that I rebased and force-pushed most of the open PR branches to update the CI which is now running through GitHub Actions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
This is a backwards compatible enhancement:
For discrete and quasi-continuous control it is essential to sample the current at the right time instant during the pwm period to catch the mean current. Using triangle reference signal, it is easy to sample exactly in the middle of the pwm period.