Opened 5 years ago

Closed 2 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: ceraolo Owned by: adrpo
Priority: blocker Milestone: 1.19.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 5 years ago.
media.mos​ (120 bytes) - added by adeas31 5 years ago.
Another crash case

Download all attachments as: .zip

Change History (18)

comment:1 Changed 5 years 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 5 years ago by adeas31

  • Status changed from new to accepted

comment:3 Changed 5 years 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 5 years ago by adeas31

comment:4 Changed 5 years 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 5 years ago by adeas31

Another crash case

comment:6 Changed 5 years 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 4 years 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 4 years 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 4 years 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 4 years 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 4 years 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 3 years ago by casella

  • Milestone changed from 1.16.0 to 1.17.0

Retargeted to 1.17.0 after 1.16.0 release

comment:13 Changed 3 years ago by casella

  • Milestone changed from 1.17.0 to 1.18.0

Rescheduled to 1.18.0

comment:14 Changed 3 years ago by casella

  • Milestone 1.18.0 deleted

Ticket retargeted after milestone closed

comment:15 Changed 3 years ago by casella

  • Milestone set to 1.19.0

1.18.0 blocker tickets moved to 1.19.0

comment:16 Changed 2 years ago by perost

  • Resolution set to fixed
  • Status changed from assigned to closed

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.