Opened 6 years ago

Closed 6 years ago

Last modified 4 years ago

#5400 closed defect (fixed)

Moving a model in package breaks other models

Reported by: massimo ceraolo Owned by: Per Östlund
Priority: blocker Milestone: 1.14.0
Component: Interactive Environment Version: v1.14.0-dev-nightly
Keywords: Cc: Adrian Pop, Per Östlund

Description (last modified by massimo ceraolo)

to reproduce the issue both mo files must be loaded and the txt file should be added in the OMEdit working directory.

Once both mo files are loaded
1) open myEVPkg.WithRecord.EVBasicMB.
2) Run this model -> everything it ok.
3) click on "Car", and move it downwards through ctrl+down arrow.
4) run again EVBasicMB -> it does not run anymore!

Tested with OM 1.14-dev-383

Attachments (4)

AccBrake.txt (97 bytes ) - added by massimo ceraolo 6 years ago.
myEVpkg.mo (17.0 KB ) - added by massimo ceraolo 6 years ago.
wbEHPTLib.mo (185.7 KB ) - added by massimo ceraolo 6 years ago.
5400.mos (221 bytes ) - added by Adeel Asghar 6 years ago.

Download all attachments as: .zip

Change History (12)

by massimo ceraolo, 6 years ago

Attachment: AccBrake.txt added

by massimo ceraolo, 6 years ago

Attachment: myEVpkg.mo added

by massimo ceraolo, 6 years ago

Attachment: wbEHPTLib.mo added

comment:1 by Francesco Casella, 6 years ago

@ceraolo, I'm not sure about the version number, it seems the Linux one, which doesn't match the Windows one for some reason. Are you using the latest nightly? There were some fixes pushed in by @adeas31 two days ago on possibly related issues.

comment:2 by massimo ceraolo, 6 years ago

It is today's nightly build under linux.
The same issue is seen using the 1.13.1 version under windows.

comment:3 by Adeel Asghar, 6 years ago

Cc: Adrian Pop Per Östlund added
Component: OMEditInteractive Environment
Owner: changed from Adeel Asghar to somebody

@casella my changes has nothing to do with moving of the model.
@ceraolo I looked at it very briefly. The problem is that moveClass API is somehow clearing the whole AST. A made a mos-script that shows the problem.

This is a very serious bug we need to fix this ASAP. I am making it a blocker.

Last edited 6 years ago by Adeel Asghar (previous) (diff)

by Adeel Asghar, 6 years ago

Attachment: 5400.mos added

comment:4 by Adeel Asghar, 6 years ago

Priority: highblocker

comment:5 by Francesco Casella, 6 years ago

Owner: changed from somebody to Per Östlund
Status: newassigned

@perost, can you have a look at this while @adrpo is busy with the new API?

in reply to:  5 comment:6 by Per Östlund, 6 years ago

Resolution: fixed
Status: assignedclosed

Replying to casella:

@perost, can you have a look at this while @adrpo is busy with the new API?

It looks like adeas31 proposed a fix shortly after the issue was reported, which I then improved and implemented in 640f0fc. But it seems neither of us remembered to update the ticket.

comment:7 by Francesco Casella, 6 years ago

OK, thanks!

comment:8 by massimo ceraolo, 4 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.