Bug in a component: Modelica.Electrical.Machines.Sensors.HallSensor

[ankitnaik177]

The block Modelica.Electrical.Machines.Sensors.HallSensor has an error. It uses rem() but it really should be mod().

The hall sensor should always be between 0 and 2*pi — no matter the direction of travel. The electrical angle of a motor is always in that range — as soon as you reach 2 Pi you wrap back to zero and if you pass zero in the negative direction, you wrap to 2 pi. Note that if the angle is negative, the HallSensor gives a nonsensical output.

[dietmarw]

@AHaumer To me, it looks like that @ankitnaik177 is right. Taking the example Modelica.Electrical.Machines.Examples.SynchronousMachines.SMPM_NoLoad and letting the machine run backwards will create the following angle outputs for the original HallSensor(Rem) and a modified using mod

[dietmarw]

@AHaumer How severe do you reckon this fault is? Does it justify backporting to 4.0.x branch?

[AHaumer]

To be honest: the difference between the two variants is 2*pi - so I wouldn't care for backporting.

[dietmarw]

OK. In that case, I'll remove the "bug" label from #4031.