Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#2627 closed defect (fixed)

OMEdit crashes under Windows

Reported by: Rüdiger Franke Owned by: Adeel Asghar
Priority: high Milestone: 1.9.1
Component: OMEdit Version: trunk
Keywords: Cc: Adrian Pop, per, Martin Sjölund

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 Rüdiger Franke 11 years ago.
openmodelica.omc.output.OMEdit (390 bytes ) - added by Rüdiger Franke 11 years ago.
ClickClickCrash.png (87.2 KB ) - added by Rüdiger Franke 11 years ago.

Change History (15)

comment:1 by Adeel Asghar, 11 years ago

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

by Rüdiger Franke, 11 years ago

Attachment: omeditcommands.log added

by Rüdiger Franke, 11 years ago

comment:2 by Rüdiger Franke, 11 years ago

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

comment:3 by Adeel Asghar, 11 years ago

Cc: Adrian Pop per Martin Sjölund 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 by Adrian Pop, 11 years ago

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 Adrian Pop (previous) (diff)

comment:5 by Adrian Pop, 11 years ago

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

comment:6 by Adrian Pop, 11 years ago

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

comment:7 by Rüdiger Franke, 11 years ago

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 :-)

by Rüdiger Franke, 11 years ago

Attachment: ClickClickCrash.png added

comment:8 by Rüdiger Franke, 11 years ago

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 by Adeel Asghar, 11 years ago

Resolution: fixed
Status: newclosed

comment:10 by Martin Sjölund, 11 years ago

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

comment:11 by Adeel Asghar, 11 years ago

Adrian fixed it r19647.

comment:12 by Martin Sjölund, 11 years ago

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.