Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#5844 closed defect (duplicate)

Repeated OMEditor Crashes

Reported by: anonymous Owned by: adeas31
Priority: blocker Milestone: 1.16.0
Component: Interactive Environment Version: v1.14.1
Keywords: Crash Cc: adrpo

Description

OpenModelica Connection Editor crashes every time Modelica.Media.Air is expanded in the libraries browser.

Change History (10)

comment:1 Changed 5 years ago by casella

  • Cc adrpo added
  • Component changed from *unknown* to Interactive Environment
  • Milestone changed from Future to 1.14.2
  • Owner changed from somebody to adeas31
  • Priority changed from high to blocker
  • Status changed from new to assigned

This is quite bad, we'll try to fix it for 1.15.0 or in a second 1.14.2 maintenance release if 1.15.0 is further delayed.

In fact, in my case it just hangs indefinitely.

@adeas31, can you quickly asses where does this go wrong? I have changed the OMEdit working directory, and I can no longer find the omcommunication file (where is it?)

comment:2 follow-up: Changed 5 years ago by adeas31

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

This is the same issue as reported in #5397 and #5575.

@casella the communication file is always generated in the temp directory regardless of the working directory.

comment:3 in reply to: ↑ 2 Changed 5 years ago by casella

Replying to adeas31:

This is the same issue as reported in #5397 and #5575.

I'm not sure about the latter. One thing is being slow, another thing is hanging indefinitely.

@casella the communication file is always generated in the temp directory regardless of the working directory.

The question is, where is the temp directory? For me, it used to be under C:\Users\Francesco Casella\AppData, which is incidentally a hidden directory (bravo! to Microsoft for making things easy to manage). But I cannot find it there any more now that I moved the working directory elsewhere.

comment:4 follow-up: Changed 5 years ago by adeas31

omeditcommunication.log file is always in %TEMP%/OpenModelica/OMEdit. Do you by any chance has changed your Windows temp location?

comment:5 in reply to: ↑ 4 Changed 5 years ago by casella

Replying to adeas31:

omeditcommunication.log file is always in %TEMP%/OpenModelica/OMEdit.

There, I was looking for this information but I couldn't find it out, I forgot the Temp after AppData\Local. Could we maybe add a final short section at the end of the OMEdit documentation mentioning these log files and their location? I think this could be useful to other people too.

comment:6 Changed 5 years ago by casella

Last lines in omeditcommands.mos:

getInheritedClasses(Modelica.Media.Air.ReferenceAir); getErrorString();
errors:=getMessagesStringInternal(); getErrorString();
size(errors,1); getErrorString();
getNthInheritedClassIconMapAnnotation(Modelica.Media.Air.ReferenceAir, 1); getErrorString();
getNthInheritedClassDiagramMapAnnotation(Modelica.Media.Air.ReferenceAir, 1); getErrorString();
getIconAnnotation(Modelica.Media.Air.ReferenceAir); getErrorString();
errors:=getMessagesStringInternal(); getErrorString();
size(errors,1); getErrorString();
getInheritedClasses(Modelica.Media.Air.MoistAir); getErrorString();
errors:=getMessagesStringInternal(); getErrorString();
size(errors,1); getErrorString();
getInheritedClasses(Modelica.

I guess the Windows buffer was not flushed properly. Ditto for omeditcommunication.log:

getInheritedClasses(Modelica.Media.Air.ReferenceAir) 17:10:19:487
{Modelica.Icons.VariantsPackage} 17:10:19:497
#s#; 0.010000; 10.418000; 'getInheritedClasses(Modelica.Media.Air.ReferenceAir)'

errors:=getMessagesStringInternal() 17:10:19:497
{} 17:10:19:501
#s#; 0.004000; 10.422000; 'errors:=getMessagesStringInternal()'

size(errors,1) 17:10:19:501
0 17:10:19:502
#s#; 0.001000; 10.423000; 'size(errors,1)'

getNthInheritedClassIconMapAnnotation(Modelica.Media.Air.ReferenceAir, 1) 17:10:19:502
{Modelica.Icons.VariantsPackage,{}} 17:10:19:502
#s#; 0.000000; 10.423000; 'getNthInheritedClassIconMapAnnotation(Modelica.Media.Air.ReferenceAir, 1)'

getNthInheritedClassDiagramMapAnnotation(Modelica.Media.Air.ReferenceAir, 1) 17:10:19:502
{Modelica.Icons.VariantsPackage,{}} 17:10:19:502
#s#; 0.000000; 10.423000; 'getNthInheritedClassDiagramMapAnnotation(Modelica.Media.Air.ReferenceAir, 1)'

getIconAnnotation(Modelica.Media.Air.ReferenceAir) 17:10:19:503
{} 17:10:19:503
#s#; 0.000000; 10.423000; 'getIconAnnotation(Modelica.Media.Air.ReferenceAir)'

errors:=getMessagesStringInternal() 17:10:19:503
{} 17:10:19:507
#s#; 0.004000; 10.427000; 'errors:=getMessagesStringInternal()'

size(errors,1) 17:10:19:507
0 17:10:19:509
#s#; 0.002000; 10.429000; 'size(errors,1)'

getInheritedClasses(Modelica.Media.Air.MoistAir) 17:10:19:512
{Modelica.Media.Interfaces.PartialCondensingGases} 17:10:21:784
#s#; 2.272000; 12.701000; 'getInheritedClasses(Modelica.Media.Air.MoistAir)'

errors:=getMessagesStringInternal() 17:10:21:784
{} 17:10:21:797
#s#; 0.013000; 12.714000; 'errors:=getMessagesStringInternal()'

size(errors,1) 17:10:21:797
0 17:10:21:798
#s#; 0.001000; 12.715000; 'size(errors,1)'

getInheritedClasses(Modelica.Media.Interfaces.PartialCondensingGases) 17:10:21:800
{Modelica.Media.Interfaces.PartialMixtureMedium} 17:10:21:831
#s#; 0.031000; 12.746000; 'getInheritedClasses(Modelica.Media.Interfaces.PartialCondensingGases)'

errors:=getMessagesStringInternal() 17:10:21:831
{} 17:10:21:835
#s#; 0.003000; 12.749000; 'errors:=getMessagesStringInternal()'

size(errors,1) 17:10:21:835
0 17:10:21:836
#s#; 0.001000; 12.750000; 'size(errors,1)'

getInheritedClasses(Modelica.Media.Interfaces.PartialMixtureMedium) 17:10:21:837
{Modelica.Media.Interfaces.PartialMedium} 17:10:21:864
#s#; 0.027000; 12.777000; 'getInheritedClasses(Modelica.Media.Interfaces.PartialMixtureMedium)'

errors:=getMessagesStringInternal() 17:10:21:865
{} 17:10:21:868
#s#; 0.003000; 12.780000; 'errors:=getMessagesStringInternal()'

size(errors,1) 17:10:21:868
0 17:10:21:870
#s#; 0.002000; 12.782000; 'size(errors,1)'

getNthInheritedClassIconMapAnnotation(

comment:7 Changed 5 years ago by casella

It seems like something goes wrong when handling either Air.MoistAir or Air.ReferenceMoistAir

comment:8 Changed 5 years ago by adeas31

Yes, that is because of getInheritedClasses(Modelica.Media.Air.ReferenceMoistAir). See attachment:media.mos:ticket:5397

There, I was looking for this information but I couldn't find it out, I forgot the Temp after AppData\Local. Could we maybe add a final short section at the end of the ​OMEdit documentation mentioning these log files and their location? I think this could be useful to other people too.

@casella can you make a separate ticket about it so I don't miss it.

comment:9 Changed 5 years ago by casella

See #5851

comment:10 Changed 4 years ago by casella

  • Milestone changed from 1.14.2 to 1.16.0
Note: See TracTickets for help on using tickets.