Modify

Opened 21 months ago

Last modified 5 weeks ago

#5397 assigned defect

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

Reported by: ceraolo Owned by: adrpo
Priority: blocker Milestone: 1.17.0
Component: Interactive Environment Version: v1.13.0
Keywords: Cc: perost

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 adeas31 21 months ago.
media.mos (120 bytes) - added by adeas31 20 months ago.
Another crash case

Download all attachments as: .zip

Change History (14)

comment:1 Changed 21 months ago by casella

  • Priority changed from high to blocker

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 Changed 21 months ago by adeas31

  • Status changed from new to accepted

comment:3 Changed 21 months ago by adeas31

  • Cc perost added
  • Component changed from OMEdit to Interactive Environment
  • Owner changed from adeas31 to adrpo
  • Status changed from accepted to assigned

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

Changed 21 months ago by adeas31

comment:4 Changed 21 months ago by adrpo

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.

Changed 20 months ago by adeas31

Another crash case

comment:6 Changed 19 months ago by casella

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

comment:7 Changed 14 months ago by casella

  • Milestone changed from 1.14.0 to 1.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 Changed 6 months ago by casella

  • Milestone changed from 1.15.0 to 1.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 Changed 3 months ago by Michael Schuh <OpenModelica@…>

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

comment:10 follow-up: Changed 3 months ago by Michael Schuh <OpenModelica@…>

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

comment:11 in reply to: ↑ 10 Changed 3 months ago by casella

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 Changed 5 weeks ago by casella

  • Milestone changed from 1.16.0 to 1.17.0

Retargeted to 1.17.0 after 1.16.0 release

Add Comment

Modify Ticket

Change Properties
Set your email in Preferences
Action
as assigned The owner will remain adrpo.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from adrpo to the specified user. Next status will be 'assigned'.
The owner will be changed from adrpo to anonymous. Next status will be 'accepted'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.