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.

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

LeakageWithCoefficient should have a port for reluctance input #3300

Closed
JoeRiel opened this issue Jan 10, 2020 · 5 comments · Fixed by #3307 or #3431
Closed

LeakageWithCoefficient should have a port for reluctance input #3300

JoeRiel opened this issue Jan 10, 2020 · 5 comments · Fixed by #3307 or #3431
Assignees
Labels
discussion Discussion issue that it not necessarily related to a concrete bug or feature L: Magnetic.FluxTubes Issue addresses Modelica.Magnetic.FluxTubes
Milestone

Comments

@JoeRiel
Copy link
Contributor

JoeRiel commented Jan 10, 2020

The model Modelica.Magnetic.FluxTubes.Basic.LeakageWithCoefficient has an input signal, R_mUsefulTot, that provides the total reluctance of the path in which it is connected. To make this part usable in a GUI, that input should be a port to which a real signal can be connected.

Actually, it's not clear to me why this quantity is an input and not a parameter. Are there examples of a model where it varies? The doc-string for the component ("... not for dynamic simulation of actuators") suggests it isn't intended to be used in that application.

@beutlich beutlich added the L: Magnetic.FluxTubes Issue addresses Modelica.Magnetic.FluxTubes label Jan 12, 2020
@AHaumer
Copy link
Contributor

AHaumer commented Jan 12, 2020

I'm not sure whether input should be a parameter, I'll ask the authors about their intentions.
"... not for dynamic simulation of actuators": Please read the full doc, the authors wrote:
" ...This would generate a reluctance force in the leakage element that is not accounted for properly."
I interpret this thay way that if you don'ct care about the reluctance force you can use this leakage element in parallel to the "useful" element, assuming that the coupling coefficient remains constant which in turns leads to a change of the leakage reluctance if "useful" reluctance changes. Following the idea, R_mUsefulTot should be an input rather than a parameter. Dealing with saturation in the "useful" element, reluctance changes with point of operation.
Additionally, I'd set limits for eps<c_usefulFlux<1-eps

@beutlich beutlich added the discussion Discussion issue that it not necessarily related to a concrete bug or feature label Jan 12, 2020
AHaumer added a commit to AHaumer/ModelicaStandardLibrary that referenced this issue Jan 12, 2020
@AHaumer
Copy link
Contributor

AHaumer commented Jan 12, 2020

I've created a PR, showing the usage in the Example BasicExamples.QuadraticCoreAirgap

@AHaumer AHaumer added this to the MSL4.0.0 milestone Jan 12, 2020
@beutlich beutlich changed the title LeakageWithCoefficent should have a port for reluctance input LeakageWithCoefficient should have a port for reluctance input Jan 13, 2020
beutlich added a commit that referenced this issue Jan 14, 2020
@christiankral
Copy link
Contributor

Re-open due to #3424 (comment)

@dietmarw
Copy link
Member

Fixed again by #3424

@dietmarw
Copy link
Member

Probably wait for #3431 first...

@dietmarw dietmarw reopened this Feb 18, 2020
dietmarw pushed a commit that referenced this issue Feb 20, 2020
…rt (#3431)

* Change parameter c_usefulFlux default to start
Fixes #3300
Refs #3424

* Add conversion script testing #3300 and #3424
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Discussion issue that it not necessarily related to a concrete bug or feature L: Magnetic.FluxTubes Issue addresses Modelica.Magnetic.FluxTubes
Projects
None yet
5 participants