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

Weird connect equations present in MSL3.2 and some user models #642

Closed
modelica-trac-importer opened this issue Jan 14, 2017 · 8 comments
Closed
Assignees
Labels
bug Critical/severe issue L: Electrical.Machines Issue addresses Modelica.Electrical.Machines
Milestone

Comments

@modelica-trac-importer
Copy link

Reported by adrpo on 20 Nov 2011 08:51 UTC

Hi all,

There are several connect equations that connect the same
connector instance, i.e. connect(A, A) present in MSL 3.2
(we also found some of these in some user models).

Question:
Is there some voodoo going on or these are just no-ops?
There's nothing in the specification about something like this.
Currently we just issue a warning and ignore them.

For example:

[Modelica 3.2/Electrical/QuasiStationary/MultiPhase.mo:1872:7-1873:76:writable] Warning: connect(currentP, currentP) connects the same connector instance! The connect equation will be ignored.
[Modelica 3.2/Electrical/Digital.mo:5309:11-5312:35:writable] Warning: connect(dataOut, dataOut) connects the same connector instance! The connect equation will be ignored.
[Modelica 3.2/Electrical/Digital.mo:6206:11-6209:35:writable] Warning: connect(dataOut, dataOut) connects the same connector instance! The connect equation will be ignored.
[Modelica 3.2/Electrical/Digital.mo:6678:11-6681:35:writable] Warning: connect(dataOut, dataOut) connects the same connector instance! The connect equation will be ignored.
[Modelica 3.2/Electrical/Machines.mo:3880:9-3883:33:writable] Warning: connect(plug_rn, plug_rn) connects the same connector instance! The connect equation will be ignored.

Cheers,
Adrian Pop/


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

@modelica-trac-importer
Copy link
Author

Comment by pharman on 20 Nov 2011 10:41 UTC
See also #552, though that doesn't answer the question of why they are there to start with.

@modelica-trac-importer
Copy link
Author

Comment by adrpo on 20 Nov 2011 11:05 UTC

Thanks for the pointer Peter.
So there is some voodoo going on after all :)
Yet another classic special case that we're always glad to discover.
Well, at least now I know how we should handle them.

Cheers,
Adrian Pop/

@modelica-trac-importer
Copy link
Author

Comment by volker.beuter on 20 Nov 2011 23:29 UTC
This is not a new problem. In a project working with customer libraries we are struggling
with lots of those self connections, mostly on data busses. No one says he had done such
connections purposely. These libraries have been created with Dymola version 6.1 and earlier
so I suspect it must have been easy to create self connections accidently with these versions
and there have been such self connections also in earlier MSL versions.
Tools ought to give at least a warning in case such a connections has been created
during graphical composition of a model. For example Dymola 2012 does not give any warning
on this - but here I had to create it on purpose. (This is of course no matter of the
specification.)

@modelica-trac-importer
Copy link
Author

Comment by majetta on 21 Nov 2011 09:00 UTC
The connections in

[Modelica 3.2/Electrical/Digital.mo:5309:11-5312:35]
[Modelica 3.2/Electrical/Digital.mo:6206:11-6209:35] 
[Modelica 3.2/Electrical/Digital.mo:6678:11-6681:35]

are removed in c60c4eb.

@modelica-trac-importer
Copy link
Author

Modified by dietmarw on 21 Nov 2011 09:39 UTC

@modelica-trac-importer
Copy link
Author

Comment by anonymous on 22 Nov 2011 09:08 UTC
Could you please fix MSL3.2 maintenance branch, too!

@modelica-trac-importer
Copy link
Author

Comment by dietmarw on 22 Nov 2011 12:58 UTC
Fixed for Machines.mo and MultiPhase.mo in 7a502aa

@modelica-trac-importer
Copy link
Author

Comment by dietmarw on 22 Nov 2011 13:39 UTC
Fixed in 84f7ae8 : 6d9780a for 3.2 maintenance.

@modelica-trac-importer modelica-trac-importer modified the milestone: MSL3.2.1 Jan 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Critical/severe issue L: Electrical.Machines Issue addresses Modelica.Electrical.Machines
Projects
None yet
Development

No branches or pull requests