[HowTo] FindFailed handler: General error handling --- 1.1.3 +

ok, understood.

I have to check. May take some hours ;-)

ok, the situation is a bit complicated and the current docs do not reflect that sufficiently, sorry.

In short for Python scripting:
- setFindFailedHandler and setFindFailedResponse are Region specific, which means they are only used, for the Region, they are specified for, when a search/FindFailed happens.
- when a FindFailed situation happens, first the handler is visited, if specified for the region object. Then the setting of the FindFailedResponse of that region object is obeyed (which in turn might have been modified before in a handler)
- response and/or handler can be specified globally (FindFailed. setFindFailedResponse and FindFailed. setFindFailedResponse), so that every region object created afterwards will inherit the global settings

I will revise the docs accordingly asap.

For Python scripting, there is one special thing to be aware:
see bug 1778855

Since I am not using any regions at all I just assumed it would work for the whole screen. To be honest, I have not yet wrapped my hread around the concept of regions.

I already tried using FindFailed.setFindFailedHandler and the shortcuts for it mentioned in the docs. However, I get an error message if I do so. Not at the right PC now, but IIRC it said something about FindFailed does not have this attribute...will try to reproduce the error tonight.

Just to be sure: do I have to start using regions if I want to use setFindFailedHandler? I was really hoping that I don't have to dive into this.

Accidentally clicked on "this solved my problem"

--- do I have to start using regions if I want to use setFindFailedHandler?
No, you do not have to.

See the mentioned bug for a workaround.

I will again check with version 1.1.2 (I normally work with the latest 1.1.3 nightly)

ok, this is due to the not-doced-problem:

FindFailed.setResponse(...)
FindFailed.setHandler(...)

and
FindFailed.getResponse()

are the respective methods.

To be on the safe side with the current implementation, you should act so:

def ffhandler(event):
  print "in FindFailed handler"

FindFailed.setHandler(ffhandler) # set as global handler even for Regions
use() # reset the global SCREEN object to take over the handler setting

This must be inserted at the top of the main script.

If you want this behavior in any imported script (having "from sikuli import *" at the beginning), then those scripts need this also.

If you want to share the handler among scripts, then it must be in one script

# FindFailedHandling.sikuli
from sikuli import *
def ffhandler(event):
  print "in FindFailed handler"

and then in the script that needs it:
from sikuli import *
from FindFailedHandling import *
FindFailed.setHandler(ffhandler)
use()

in the main script the
from sikuli import *
is not needed

I am using a single script for now. With the method you proposed in your last post, the handler routine is still not invoked on FindFailed. The script still aborts when FindFailed occurs.

Kind of got it working by putting these lines on top of the script:
def ffhandler(event):
    resetGUI()
FindFailed.setHandler(ffhandler)
use()
setFindFailedHandler(ffhandler)

However I was surprised to learn that apparently even constructs like these
while(not exists(Pattern("1527938243237-3.png").similar(0.95),1.0)):
        #do stuff
count as a FindFailed. At least they invoke the handler script which renders all of this pretty useless for what I had in mind.

def ffhandler(event):
    resetGUI()
FindFailed.setHandler(ffhandler)
use()

... should be sufficient

--- exists()
accepted - I have to find a way, to fix it.
I think, that the best thing would be, that the handler should know, wether it was an exists, so you can decide, to do nothing.
To generally switch of the visit of the handler with exists does not make sense, since other things than your idea a user might want to implement in the handler.

I will be back within 24 hours.

That is exactly what I tried first. Did not invoke the handler script on a FindFailed.

I tend to disagree with you on the exsts() issue. Unlike other features like find or click, it does not abort the script with a FindFailed if the image is not found.

--- Did not invoke the handler
ok, I have to check again and fix if needed

--- I tend to disagree with you on the exists() issue
I tried to find things, that supports my view - but found none.
So I will now switch off FindFailed handling with exists() completely and wait if someone complains about in the future.

Thanks for your cooperation.

Thank you for your support and this awesome piece of software. Despite the issues i am still facing, it allowed me to do much more and in a shorter amount of time than any other automation software that I am aware of.

Thanks RaiMan, that solved my question.

Works like a charm with the latest build. Dankeschön!

Alles Gute ;-)