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

Bug in Modelica.Electrical.Analog.Ideal.ControlledIdealIntermediateSwitch #396

Closed
modelica-trac-importer opened this issue Jan 14, 2017 · 3 comments
Closed

Bug in Modelica.Electrical.Analog.Ideal.ControlledIdealIntermediateSwitch #396

modelica-trac-importer opened this issue Jan 14, 2017 · 3 comments
Assignees
@ECClauss
Labels
bug Critical/severe issue L: Electrical.Analog Issue addresses Modelica.Electrical.Analog
Milestone
MSL3.2

Comments

@modelica-trac-importer
Copy link

Reported by stefanv on 9 Aug 2010 19:34 UTC
It appears that the documentation and the code for Modelica.Electrical.Analog.Ideal.ControlledIdealIntermediateSwitch are in disagreement with one another. The documentation (especially the diagrams) suggests that when control.v > level, the switch is connected straight through, and when control.v <= level, the switch is connected in a cross-over. Solving the equations in the code seems to suggest the opposite.

It also appears that someone started writing the text of the documentation correctly, and then changed their mind half way through the sentence: "p1 is connected to n2, and p2 is connected to n2".

Migrated-From: https://trac.modelica.org/Modelica/ticket/396
@modelica-trac-importer modelica-trac-importer added the bug Critical/severe issue label Jan 14, 2017
@modelica-trac-importer
Copy link
Author

Modified by dietmarw on 10 Aug 2010 05:54 UTC

@modelica-trac-importer modelica-trac-importer added the L: Electrical.Analog Issue addresses Modelica.Electrical.Analog label Jan 14, 2017
@modelica-trac-importer modelica-trac-importer added this to the MSL3.2 milestone Jan 14, 2017
@modelica-trac-importer
Copy link
Author

Comment by Christoph.Clauss on 17 Aug 2010 13:12 UTC
It is correct: The documentation was wrong.
The documentation was improved on the trunk(MSL3.2) a8e9d43, and on maintenance MSL 3.1 49b0b9a, 4bd542d.
The switch behaviour was not changed.

@modelica-trac-importer
Copy link
Author

Modified by dietmarw on 17 Aug 2010 13:34 UTC

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

@ECClauss ECClauss

Labels
bug Critical/severe issue L: Electrical.Analog Issue addresses Modelica.Electrical.Analog
Projects
None yet
Milestone
MSL3.2
Development

No branches or pull requests
3 participants
@MartinOtter @modelica-trac-importer @ECClauss