Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#2627 closed defect (fixed)

OMEdit crashes under Windows

Reported by: rfranke Owned by: adeas31
Priority: high Milestone: 1.9.1
Component: OMEdit Version: trunk
Keywords: Cc: adrpo, per, sjoelund.se

Description

Using 1.9.1b2 or nightly 3/18, OMEdit crashes under Windows 7 Pro with:

  1. start OMEdit.exe
  2. open Modelica library by clicking on respective +-sign in Libraries pane -- the package icons are just grey without graphics!?
  3. open Blocks by clicking on the respective +-sign
  4. open Continuous by clicking on the respective +-sign

OMEdit.exe crashes.

Doing the same under Debian Wheezy, the package icons are filled with the respective graphics and no crash occurs.

Attachments (3)

omeditcommands.log (3.0 MB) - added by rfranke 11 years ago.
openmodelica.omc.output.OMEdit (390 bytes) - added by rfranke 11 years ago.
ClickClickCrash.png (87.2 KB) - added by rfranke 11 years ago.

Change History (15)

comment:1 Changed 11 years ago by adeas31

I can't reproduce this error on Windows 7 Professional.

I need log files,
%TEMP%/OpenModelica/OMEdit/omeditcommands.log
%TEMP%/OpenModelica/OMEdit/openmodelica.omc.output.OMEdit

Changed 11 years ago by rfranke

Changed 11 years ago by rfranke

comment:2 Changed 11 years ago by rfranke

The crash repeated with nightly 3/19. Find attached the requested log files.

comment:3 Changed 11 years ago by adeas31

  • Cc adrpo per sjoelund.se added

I see that you have +a flag enabled.

+a, +keepArrays            Sets whether to split arrays or not.

This is changing the result of all annotation APIs. I am not sure if this flag is suppose to change the results. I have added few more developers in the CC list how might know more.

If you don't need this flag then simply run OMEdit and then go to Tools->Options->Simulation and set OMC Flags to empty.

comment:4 Changed 11 years ago by adrpo

The +a flag is rather experimental and should not be used by default.
Especially it should not be used for anything but flattening.
It doesn't think it will even work for simulation.

Last edited 11 years ago by adrpo (previous) (diff)

comment:5 Changed 11 years ago by adrpo

I guess we should disable the +a flag at least when we evaluate annotations.

comment:6 Changed 11 years ago by adrpo

I disabled +a when dealing with annotations in r19647.
This might fix the issue, but really +a should not be used interactively.

comment:7 Changed 11 years ago by rfranke

Disabling +a in OMEdit seems to solve all my issues:

  • The package icons show in the Libraries pane
  • Browsing to Modelca.Blocks.Continuous works
  • Check of AlgorithmicModel.BaseModelAlgorithm is still fast
  • I can simulate the examples in AlgorithmicModel

Great. Thank you!

(Just translation of the models is slow -- looking forward to eventually seeing some improved +a flag :-)

Changed 11 years ago by rfranke

comment:8 Changed 11 years ago by rfranke

Just for completeness: ClickClickCrash.png shows a screenshot with +a flag: the package icons don't show up in the Modelica library and attempting to browse into Modelica.Blocks.Continuous leads to the crash (read from memory location 0 in QtGui4.dll).

comment:9 Changed 11 years ago by adeas31

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

comment:10 Changed 11 years ago by sjoelund.se

Did you fix the crash if +a was enabled, Adeel?

comment:11 Changed 11 years ago by adeas31

Adrian fixed it r19647.

comment:12 Changed 11 years ago by sjoelund.se

That still does not fix OMEdit crashing if it is fed improper input... It must never crash regardless what stupid things omc sends back.

Note: See TracTickets for help on using tickets.