Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#3657 closed defect (duplicate)

OMEdit no longer renders icons

Reported by: Dietmar Winkler Owned by: Adeel Asghar
Priority: critical Milestone: 1.11.0
Component: OMEdit Version:
Keywords: Cc: Adrian Pop

Description

In the latest beta.1 OMEdit stopped rendering icons in the diagram layer (might have been the case earlier).

Just compare ModelicaByExample.Architectures.ThermalControl.Examples.OnOffVariant as it should look and generated by OM's Icon2SVG renderer:

http://book.xogeny.com/_images/OnOffVariant.svg

vs OMEdits display:


Attachments (1)

OnOff.png (2.3 KB ) - added by Dietmar Winkler 9 years ago.

Download all attachments as: .zip

Change History (11)

by Dietmar Winkler, 9 years ago

Attachment: OnOff.png added

comment:1 by Adeel Asghar, 9 years ago

Cc: Adrian Pop added

This can't be a svg generated by OM since we don't have any API to query the redeclare replaceable stuff from model.

There are already tickets about that missing feature like #2850, #2479 and #2079 but we can keep this one also as this one contains a simple example as well.

comment:2 by Dietmar Winkler, 9 years ago

Well it is an svg that is generated from the python script that ships with OM ;-)
https://github.com/OpenModelica/OMCompiler/blob/master/Examples/generate_icons.py

comment:3 by Dietmar Winkler, 9 years ago

Ah forget my comment above. I was mixing the icon generation with the diagram images.

comment:4 by Adeel Asghar, 9 years ago

Milestone: 1.9.4Future
Priority: blockercritical

comment:5 by Dietmar Winkler, 8 years ago

Any chance to get this looked at for the next release?

comment:6 by Dietmar Winkler, 8 years ago

#3706 seems to be the same issue.

in reply to:  6 comment:7 by Adeel Asghar, 8 years ago

Replying to dietmarw:

#3706 seems to be the same issue.

No. In #3706 the problem is the use of undeclared parameter. The problem in this ticket is that we don't support redeclare replaceable at the API level.

comment:8 by Francesco Casella, 8 years ago

Resolution: duplicate
Status: newclosed

Seems like a duplicate of #2850. Please reopen if not.

comment:9 by Dietmar Winkler, 8 years ago

@casella, yes you seem to be right here. Well spotted.

comment:10 by Adeel Asghar, 8 years ago

Milestone: Future1.11.0
Note: See TracTickets for help on using tickets.