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.

Sign up for GitHub

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

Jump to bottom

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

Open
modelica-trac-importer opened this issue Jan 15, 2017 · 6 comments
Open

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

modelica-trac-importer opened this issue Jan 15, 2017 · 6 comments
Assignees
@christiankral @AHaumer
Labels
enhancement New feature or enhancement L: Electrical.Machines Issue addresses Modelica.Electrical.Machines L: Magnetic.QuasiStatic Issue addresses Modelica.Magnetic.QuasiStatic

Comments

@modelica-trac-importer
Copy link

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 modelica-trac-importer added bug Critical/severe issue L: Electrical.Machines Issue addresses Modelica.Electrical.Machines labels Jan 15, 2017
@modelica-trac-importer
Copy link
Author

Modified by christiankral on 5 Aug 2015 20:11 UTC

@modelica-trac-importer
Copy link
Author

Modified by beutlich on 18 Aug 2015 12:06 UTC

@beutlich
Copy link
Member

Point to @JoeRiel.

@christiankral christiankral mentioned this issue Feb 17, 2017
Closed
@beutlich beutlich added this to the MSL_next-MINOR-version milestone Feb 17, 2017
@AHaumer
Copy link
Contributor

AHaumer commented Mar 4, 2017

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 beutlich added L: Magnetic.QuasiStatic Issue addresses Modelica.Magnetic.QuasiStatic enhancement New feature or enhancement and removed bug Critical/severe issue labels Mar 4, 2017
@AHaumer AHaumer modified the milestones: MSL3.2.3, MSL_next-MAJOR-version May 18, 2018
@beutlich
Copy link
Member

beutlich commented Oct 5, 2019 

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.

@AHaumer AHaumer removed this from the MSL4.0.0 milestone Jan 8, 2020
@christiankral
Copy link
Contributor

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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@christiankral christiankral

@AHaumer AHaumer

Labels
enhancement New feature or enhancement L: Electrical.Machines Issue addresses Modelica.Electrical.Machines L: Magnetic.QuasiStatic Issue addresses Modelica.Magnetic.QuasiStatic
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@dietmarw @christiankral @AHaumer @beutlich @kristinmajetta @modelica-trac-importer @ECClauss