sikuli-driver team mailing list archive

[RaiMan <question239460@xxxxxxxxxxxxxxxxxxxxx>]

Question #239460 on Sikuli changed:
https://answers.launchpad.net/sikuli/+question/239460

    Status: Open => Answered

RaiMan proposed the following answer:
--- some comments
while True:
    if not exists(adv, 0):
        break #leave the loop
    # now we check the top level actions
    if exists(adv, 0):
        click(adv)

does not really make sense: depending how the button appears and
vanishes when clicked this might either lead to excessive clicks or to a
loop lasting to long.

if you want to handle such a sequence at game start as mentioned above,
where you wait on something, then click it and then wait for the next
visual to appear, the easiest is this:

click(wait(first, 10))
click(wait(second, 10))
click(wait(third, 10))
....

the 10 is the max waiting time (if exceeded gives a FindFailed script abort) and should be set appropriate for the different steps.
In all cases: if the visual (first, second, third, ...) appears, it will be clicked immediately and the next wait() will start.

--- speed
as already mentioned in the prior comments: specifying a region smaller than the screen might reduce the search time dramatically (whole screen: 0,5 to 1 sec, small regions: down to some milli sec)

--- regions
But this is not a must from the beginning, it is more an optimisation in the second step. Only in cases, where searching the whole screen might lead to false positives , using regions might help.
But usually having good shots (as little surrounding background as possible, concentrated on the key aspects) and  using Pattern().similar(0.9) should already do the job.

Using regions:
#SRB Arena
srbregion = Region(45,35,427,643)
level = "1385933213128.png"
go = "1385933505624.png"
while True:
    if not srbregion.exists(go,0):
        break #leave the loop
    if srbregion.exists(go,0):
        click(srbregion.getLastMatch())

--- handling alternatives
If your script comes to the point, that now this or that or that might happen, again there are many possibilities to handle.
the easiest (as already mentioned in comment above)

while true:
    if exists(go, 0):
        processGo()
    if exists(battle, 0):
        processBattle()
    if exists(items, 0):
        processItems()

generally: you should go back to the above comments and implement more things to make your script more structured.
The workflow you mentioned in comment #6 is not obvious, when going through your script.

-- 
You received this question notification because you are a member of
Sikuli Drivers, which is an answer contact for Sikuli.