Opened 7 years ago

Closed 7 years ago

#4413 closed defect (fixed)

MDD models stopped working

Reported by: bthiele Owned by: ptaeuber
Priority: blocker Milestone: Future
Component: Backend Version: v1.12.0
Keywords: Cc: ptaeuber, sjoelund.se, lochel, adrpo, vwaurich

Description

There was a regression that has affected the Modelica_DeviceDrivers library.

For example, UDP communication is not working any more. I could trace the regression to the commit https://github.com/OpenModelica/OpenModelica/commit/66c68ab39d1185595d69feddaeb199c6277f49a1

Starting with the referenced commit simulation will fail (no result file is produced). It seems something goes wrong in the initialization of the external objects. Before the referenced commit a result file is produced and the external objects initialization works.

Following script can be used for testing (please check out the MDD library master branch for testing):

loadModel(Modelica);
print("*** Loading MDD...' ");
loadFile("/libraries/Modelica_DeviceDrivers/package.mo"); getErrorString();

loadString("
model RunMe
extends Modelica_DeviceDrivers.Blocks.Examples.TestSerialPackager_UDP;
end RunMe;
"); getErrorString();
simulate(RunMe); getErrorString();

I hope we can fix this regression before the beta release, since the MDD paper at the upcoming Modelica conference states that this model works now with OM.

Change History (3)

comment:1 Changed 7 years ago by ptaeuber

  • Owner changed from lochel to ptaeuber
  • Status changed from new to accepted

I will try to fix this on Monday.

comment:2 Changed 7 years ago by ptaeuber

Fixed in 9f91701. Please check if everything works fine now.

comment:3 Changed 7 years ago by bthiele

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

I just checked the UDP example and it works :-). I will do checks for the serial port etc. later this afternoon, but at first sight it looks like the issue is fixed. Thanks a lot for the quick fix!

Note: See TracTickets for help on using tickets.