specialization-examples icon indicating copy to clipboard operation
specialization-examples copied to clipboard

incorrect range of variable 'step' in escape.mzn

Open JenkinsY94 opened this issue 9 years ago • 2 comments

In line 13 of file escape.mzn , the variable is declared as: var int: step;. The scale is not precise enough. When I fixed the edge with uedge, as shown in the debugging tutorial, the model output a wrong answer and printed step = 11 rather than step = 9 as expected.

I think we should either change this line of code to var STEP: step; or add another constraint: constraint path[maxstep] = dest; in order to get the right answer.

Hope this helps. :)

JenkinsY94 avatar Oct 01 '16 12:10 JenkinsY94

On 1 Oct 2016, at 2:31 pm, JenkinsY94 [email protected] wrote:

In line 13 of file escape.mzn , the variable is declared as: var int: step;. The scale is not precise enough. When I fixed the edge with uedge, as shown in the debugging tutorial, the model output a wrong answer and printed step = 11 rather than step = 9 as expected.

I think we should either change this line of code to var STEP: step; or add another constraint: constraint path[maxstep] = dest; in order to get the right answer.

Hope this helps. :)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

I agree. But arguably this is just another bug in the initial model.

The code form escape-fixed uses the declaration

var STEP: step;

But I agree its not the point of the lecture, so I have modified the example code

pstuckey avatar Oct 03 '16 08:10 pstuckey

Related: somehow escape.dzn has an invalid character (invisible space) which is forbidden under certain circumstances in certain operating systems.

Edit: in the filename.

vitorsr avatar May 05 '19 06:05 vitorsr