Opened 6 years ago

Closed 3 years ago

#5397 closed defect (fixed)

Opening a model crashes OM 13.1 because of Inst->Static->Lookup->Inst loop started from getInheritedClasses

Reported by: massimo ceraolo Owned by: Adrian Pop
Priority: blocker Milestone: 1.19.0
Component: Interactive Environment Version: v1.13.0
Keywords: Cc: Per Östlund

Description

Using Omedit 13.1 for win (64 bit) I tried to open a model from the FCSys library :
https://github.com/kdavies4/FCSys/releases/tag/v0.2.6

My OMEdit crashes if I try to open:
FCSys.SubRegions.Examples.InternalFlow

This occurs using both old and new Front Ends

Attachments (2)

script.mos (110 bytes ) - added by Adeel Asghar 6 years ago.
media.mos (120 bytes ) - added by Adeel Asghar 6 years ago.
Another crash case

Download all attachments as: .zip

Change History (18)

comment:1 by Francesco Casella, 6 years ago

Priority: highblocker

I tried with the latest 1.14.0-dev nightly build, and in fact as soon as I tried to expand the FCSys package tree in the libraries browser, OMEdit crashes. It does it so badly that it doesn't even ask to send a bug report, just disappears from the radar screen.

I guess we should understand why this happens. As soon as @adrpo has pushed in the new API, we may also try that.

comment:2 by Adeel Asghar, 6 years ago

Status: newaccepted

comment:3 by Adeel Asghar, 6 years ago

Cc: Per Östlund added
Component: OMEditInteractive Environment
Owner: changed from Adeel Asghar to Adrian Pop
Status: acceptedassigned

Run the attached script. OMC goes into a loop and then crashes.

by Adeel Asghar, 6 years ago

Attachment: script.mos added

comment:4 by Adrian Pop, 6 years ago

Yeah, is a loop in the old front-end instantiation Inst->Static->Lookup->Inst started from getInheritedClasses. I'll see what I can do, probably we would need to use the NF for this.

by Adeel Asghar, 6 years ago

Attachment: media.mos added

Another crash case

comment:6 by Francesco Casella, 6 years ago

Summary: Open a model crashes OM 13.1Opening a model crashes OM 13.1 because of Inst->Static->Lookup->Inst loop started from getInheritedClasses

comment:7 by Francesco Casella, 5 years ago

Milestone: 1.14.01.15.0

Releasing 1.14.0 which is stable and has many improvements w.r.t. 1.13.2.

This issue, previously marked as blocker for 1.14.0, is rescheduled to 1.15.0

comment:8 by Francesco Casella, 4 years ago

Milestone: 1.15.01.16.0

Release 1.15.0 was scrapped, because replaceable support eventually turned out to be more easily implemented in 1.16.0. Hence, all 1.15.0 tickets are rescheduled to 1.16.0

comment:9 by Michael Schuh <OpenModelica@…>, 4 years ago

OpenModelica-v1.16.0-dev.03-64bit is crashing on me when I try to open the modelica.media.air library.

comment:10 by Michael Schuh <OpenModelica@…>, 4 years ago

OpenModelica-v1.16.0-dev-613-g77246831b3-64bit also crashes when I try to open the modelica.media.air library.

in reply to:  10 comment:11 by Francesco Casella, 4 years ago

Replying to Michael Schuh <OpenModelica@…>:

OpenModelica-v1.16.0-dev-613-g77246831b3-64bit also crashes when I try to open the modelica.media.air library.

This is a known issue, see #5669

comment:12 by Francesco Casella, 4 years ago

Milestone: 1.16.01.17.0

Retargeted to 1.17.0 after 1.16.0 release

comment:13 by Francesco Casella, 4 years ago

Milestone: 1.17.01.18.0

Rescheduled to 1.18.0

comment:14 by Francesco Casella, 3 years ago

Milestone: 1.18.0

Ticket retargeted after milestone closed

comment:15 by Francesco Casella, 3 years ago

Milestone: 1.19.0

1.18.0 blocker tickets moved to 1.19.0

comment:16 by Per Östlund, 3 years ago

Resolution: fixed
Status: assignedclosed

Using the new NF API (which is now the default) there doesn't seem to be any issues opening models in FCSys anymore.

Note: See TracTickets for help on using tickets.