Examples/InductionMachines/IMC_DCBraking.mo

[AHaumer]

enhances the mentioned example to avoid problems
changes the comparisonSignals (old ones are bad - see discussion in #4688)
@uschna pls. test
@GallLeo @MatthiasBSchaefer pls. prodice new reference result - thx!
@casella should we backport to 4.1.1? I suppose "no".

[AHaumer]

Sorry @uschna I have to check in detail today's evening.

[uschna]

Dear @AHaumer,
thank you for your correction. I see only few differences now, for example:

1. Placement-annotation of constantVoltage
2. shorter Types ((Modelica.Electrical.)Analog.Sources.ConstantVoltage, (Modelica.)Blocks.Sources.BooleanConstant, and (Modelica.Electrical.)Analog.Ideal.IdealOpeningSwitch)
3. other initial equations: idq_ss and idq_rs instead of idq_sr and idq_rr

Hence, I must test it again.

[AHaumer]

Thanks for looking into details, @uschna
Somehow I mixed up the old and the new version.
Should be corrected now.

[uschna]

Dear @AHaumer,
I tested it again with this version and your results. It is also valid. Thank you.

[casella]

@casella should we backport to 4.1.1? I suppose "no".

I would keep 4.1.1 for critical fixes, e.g. if we discover late that we introduced regressions in 4.1.0.

[christiankral]

Instead of comparing stator and rotor current components, we could for example compare the magnitudes of the current phasor components. In case of the stator current this is equivalent to the quasi rms current of the phase currents sensed by Polyphase.Sensors.CurrentQuasiRMSSensor:

[AHaumer]

Instead of comparing stator and rotor current components, we could for example compare the magnitudes of the current phasor components. In case of the stator current this is equivalent to the quasi rms current of the phase currents sensed by Polyphase.Sensors.CurrentQuasiRMSSensor:

Why not comparing the current phasor components, from which you are able to compute torque?

[HansOlsson]

Suggested change

	Analog.Sources.ConstantVoltage
	Modelica.Electrical.Analog.Sources.ConstantVoltage

I know that both variants work, but I find it confusing that just a few declarations use a shorter name.

[HansOlsson]

Suggested change

	Blocks.Sources.BooleanConstant booleanConstant(k=settings.layout == "Y2")
	annotation (Placement(transformation(extent={{-90,0},{-70,20}})));
	Analog.Ideal.IdealOpeningSwitch switch annotation (
	Modelica.Blocks.Sources.BooleanConstant booleanConstant(k=settings.layout == "Y2")
	annotation (Placement(transformation(extent={{-90,0},{-70,20}})));
	Modelica.Electrical.Analog.Ideal.IdealOpeningSwitch switch annotation (

Similarly.