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

Text size of icon variable #3108

Closed
max-privato opened this issue Sep 5, 2019 · 1 comment · Fixed by #3109
Closed

Text size of icon variable #3108

max-privato opened this issue Sep 5, 2019 · 1 comment · Fixed by #3109
Assignees
Labels
L: Electrical.QuasiStatic Issue addresses Modelica.Electrical.QuasiStatic
Milestone

Comments

@max-privato
Copy link
Contributor

Not the biggest problem in the world, but...
I found an inconsistency in Electrical.QuasiStationary.Basic models.

Namely, the inductance value of the inductor is much smaller than the other values in similar components (resistor, capacitor, conductor.)

In the Inductor I read:
Text(extent={{150,-50},{-150,-80}}, textString="L=%L"),

while in the Resistor, Capacitor, Conductor I read:
Text(extent={{150,-40},{-150,-80}}, textString="R=%R_ref"),
Text(extent={{150,-40},{-150,-80}}, textString="C=%C"),
Text(extent={{150,-40},{-148,-80}}, textString="G=%G_ref"),

So the fix is simple: just change -50 into -40 in the Inductor.
I suppose it is also wise to change -148 into -150 in the conductor

beutlich added a commit to beutlich/ModelicaStandardLibrary that referenced this issue Sep 5, 2019
@beutlich beutlich self-assigned this Sep 5, 2019
@beutlich beutlich added the L: Electrical.QuasiStatic Issue addresses Modelica.Electrical.QuasiStatic label Sep 5, 2019
@beutlich beutlich added this to the MSL4.0.0 milestone Sep 5, 2019
@beutlich
Copy link
Member

beutlich commented Sep 5, 2019

Not the biggest problem in the world, but...

Thanks for reporting anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: Electrical.QuasiStatic Issue addresses Modelica.Electrical.QuasiStatic
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants