Opened 11 years ago
Closed 11 years ago
#2338 closed defect (fixed)
Modelica.Electrical.QuasiStationary.MultiPhase.Examples.* are failing to compile
| Reported by: | Adrian Pop | Owned by: | Adrian Pop |
|---|---|---|---|
| Priority: | high | Milestone: | 1.9.0 |
| Component: | Frontend | Version: | trunk |
| Keywords: | Cc: |
Description
Models: BalancingStar and BalancingDelta.
This is an error in the way we deal with the breaking of overconstrained connection graph.
We build the connections sets and merge them then we delete from them the crefs which are broken.
That is wrong.
The sets should not be merged before we delete the actual connection, otherwise is impossible to know how to split the connection set.
Attachments (4)
Change History (7)
by , 11 years ago
comment:1 by , 11 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → accepted |
by , 11 years ago
Graphviz for overconstrained connection graph of BalancingDelta
by , 11 years ago
| Attachment: | BalancingStar.svg added |
|---|
by , 11 years ago
| Attachment: | BalancingDelta.svg added |
|---|
comment:2 by , 11 years ago
For example these connections for BalancingStar are broken (see also the attached graph):
Broken connections: broken(star.plugToPins_p.pin_p[2].reference, star.pin_n.reference), broken(star.plugToPins_p.pin_p[3].reference, star.pin_n.reference)
but the problem is that the set already contains many other connections
obtained via connects to other crefs (which were merged into this set)
which means that we should really not remove any of these connects but
instead split the sets.
comment:3 by , 11 years ago
| Resolution: | → fixed |
|---|---|
| Status: | accepted → closed |
Fixed in r17156. Up to Mahder now, will open a new bug.
Graphviz for overconstrained connection graph of BalancingStar