search

modelica / ModelicaStandardLibrary

Free (standard conforming) library to model mechanical (1D/3D), electrical (analog, digital, machines), magnetic, thermal, fluid, control systems and hierarchical state machines. Also numerical functions and functions for strings, files and streams are included.
https://doc.modelica.org
BSD 3-Clause "New" or "Revised" License
481 stars 169 forks source link

Modelica.Electrical.Machines.Losses.InductionMachines.PermanentMagnetLosses needs connection for stator current #1752

Open modelica-trac-importer opened 7 years ago

modelica-trac-importer commented 7 years ago

Reported by Joe Riel on 1 Aug 2015 20:50 UTC The Modelica.Electrical.Machines.Losses.InductionMachines.PermanentMagnetLosses has an input, is[m]. That should be replaced with either a real connector, or an electrical port. As it is, it is not possible to use this model in a GUI because there is no port for the stator current.

Migrated-From: https://trac.modelica.org/Modelica/ticket/1752

modelica-trac-importer commented 7 years ago

Modified by christiankral on 5 Aug 2015 20:11 UTC

modelica-trac-importer commented 7 years ago

Modified by beutlich on 18 Aug 2015 12:06 UTC

beutlich commented 7 years ago

Point to @JoeRiel.

AHaumer commented 7 years ago

Ok we need the the stator current(s) to caclulate the PM losses. There are 3 possibilites to implement that:

  1. Use physical current connectors: Looks ugly and confusing
  2. Use a current sensor and connect to a signal input of PM losses: Looks extermely ugly to use a sensor in a physical model!
  3. Use an input without visible connector (legal Modelica!): Defining the input is possbile - why do I have to connect grafically? To keep backwards compatibility, it is impossible to change the implementation.
beutlich commented 5 years ago 
3\. To keep backwards compatibility, it is impossible to change the implementation.

Backwards compatibility is not an issue for MSL v4.0.0 if a conversion can be provided.

@christiankral @AHaumer Do you think you can fix it for MSL v4.0.0? If not, please remove the milestone.

christiankral commented 4 years ago

For the component Modelica.Magnetic.FluxTubes.Basic.LeakageWithCoefficient the input was replaced by a signal input without loosing backwards compatibility, see #3300, #3307 and #3311.

@AHaumer we could still implement a similar solution this issue. Any comment?