Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#3657 closed defect (duplicate)

OMEdit no longer renders icons

Reported by: dietmarw Owned by: adeas31
Priority: critical Milestone: 1.11.0
Component: OMEdit Version:
Keywords: Cc: adrpo

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 dietmarw 9 years ago.

Download all attachments as: .zip

Change History (11)

Changed 9 years ago by dietmarw

comment:1 Changed 9 years ago by adeas31

  • Cc adrpo 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 Changed 9 years ago by dietmarw

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 Changed 9 years ago by dietmarw

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

comment:4 Changed 9 years ago by adeas31

  • Milestone changed from 1.9.4 to Future
  • Priority changed from blocker to critical

comment:5 Changed 8 years ago by dietmarw

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

comment:6 follow-up: Changed 8 years ago by dietmarw

#3706 seems to be the same issue.

comment:7 in reply to: ↑ 6 Changed 8 years ago by adeas31

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 Changed 8 years ago by casella

  • Resolution set to duplicate
  • Status changed from new to closed

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

comment:9 Changed 8 years ago by dietmarw

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

comment:10 Changed 8 years ago by adeas31

  • Milestone changed from Future to 1.11.0
Note: See TracTickets for help on using tickets.