This issue tracker is closed. Please visit UPPAAL issue tracker at Github instead.

Bug 620

Summary: optimization query breaks with time-lock where there isn't one
Product: UPPAAL Stratego Reporter: Marius Mikučionis <marius>
Component: EngineAssignee: Peter Gjøl Jensen <pgj>
Status: ASSIGNED ---    
Severity: blocker CC: fshsho7, mk09, pgj
Priority: P5    
Version: 4.1.20-3   
Hardware: All   
OS: All   
Architecture:
Attachments: stratego model demonstrating the problem (see the 1st query)
Initial state sometimes fails for optimising query with time-locked error

Description Marius Mikučionis 2016-09-29 10:03:11 CEST
Created attachment 299 [details]
stratego model demonstrating the problem (see the 1st query)

The attached model demonstrates the problem.
The problem does not appear in 4.1.20-2 version and it is not manifest for simple simulations.
This is specific to optimization query in 4.1.20-3, something is odd with evaluating possible successors when processing optimization query.
Comment 1 Martin Kristjansen 2018-02-05 13:10:38 CET
Created attachment 319 [details]
Initial state sometimes fails for optimising query with time-locked error

This model sometimes fails optimisation queries, and it fails for all version of stratego (4.1.20-*).

I have tried queries on the form E[] time < 10, so see if I can find live locks, but have not found any.
Comment 2 Peter Gjøl Jensen 2018-03-09 18:32:53 CET
There seems to be an issue with CSDecorator;

CSDecorator.maxDelay() returns inf even when CSDecorator.zeroDelay() returns true.

The problem can be fixed by replacing the maxDelay() implementation with

double maxDelay() const { return cannotDelay ? 0 : max; }

The method is only called in very few places, so I belive this fix is sound.
Comment 3 Fatima 2020-04-29 10:02:57 CEST
*** Bug 678 has been marked as a duplicate of this bug. ***
Comment 4 Marius Mikučionis 2020-10-13 13:42:13 CEST
@pgj the fix seems to have been applied, but the issue is still there.
Comment 5 Marius Mikučionis 2020-10-13 13:53:46 CEST
stratego-5 does not show this issue.
stratego-6 and stratego-7 complains