Opened 12 years ago
Closed 7 years ago
#2361 closed defect (fixed)
Bad handling of connectors derived from records
| Reported by: | Adrian Pop | Owned by: | Per Östlund |
|---|---|---|---|
| Priority: | high | Milestone: | 2.0.0 |
| Component: | Frontend | Version: | trunk |
| Keywords: | Cc: |
Description
See for example the flattened code and the generated code of
Modelica.Electrical.QuasiStationary.MultiPhase.Examples.BalancingStar
You have ComplexOutput and ComplexInput in a cast but they are not present as record constructors.
Currently we instantiate connector derived from records as records and that's not very good.
If we instantiate them as connectors then they get type:
T_COMPLEX(ClassInf.CONNECTOR(path))
and that's not handled anywhere in the back-end or so.
Somehow we should tell the compiler that this one is both
a record and a connector.
Change History (14)
comment:1 by , 12 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → accepted |
comment:2 by , 12 years ago
comment:4 by , 11 years ago
| Milestone: | 1.9.1 → 1.9.2 |
|---|
This ticket was not closed for 1.9.1, which has now been released. It was batch modified for milestone 1.9.2 (but maybe an empty milestone was more appropriate; feel free to change it).
comment:5 by , 11 years ago
| Milestone: | 1.9.2 → 1.9.3 |
|---|
Milestone changed to 1.9.3 since 1.9.2 was released.
comment:10 by , 9 years ago
| Milestone: | 1.11.0 → 1.12.0 |
|---|
Milestone moved to 1.12.0 due to 1.11.0 already being released.
comment:11 by , 8 years ago
| Milestone: | 1.12.0 → Future |
|---|
The milestone of this ticket has been reassigned to "Future".
If you think the issue is still valid and relevant for you, please select milestone 1.13.0 for back-end, code generation and run-time issues, or 2.0.0 for front-end issues.
If you are aware that the problem is no longer present, please select the milestone corresponding to the version of OMC you used to check that, and set the status to "worksforme".
In both cases, a short informative comment would be welcome.
comment:12 by , 8 years ago
| Milestone: | Future → 2.0.0 |
|---|---|
| Owner: | changed from to |
| Status: | accepted → assigned |
comment:14 by , 7 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
The Modelica.Electrical.QuasiStationary.MultiPhase.Examples.BalancingStar model now passes the simulation and verification tests, so the issue seems to have been resolved.
This is connected also to #2339 as the problem is that we get unknown
records (without declared record constructors) in the generated code.