Pythonscript show console on error

Claudia Frank

So in code.py you mean do this:

yes

You totally lost me with: “What I didn’t try is to import code and overwrite run_code function.”

In general python allows to overwrite what ever function if you use the same
function signature meaning same name and same number of arguments.
Namespaces (like what globals and locals do) could somehow try to protect function but if you try hard enough you can get it overwritten.

Depending how python script is loading code.py it could simply mean that if you
define your own runcode function you might overwrite the one from code.py.

Cheers
Claudia

Alan Kilborn

@Claudia-Frank

I tried to duplicate your “mport os” example, but I didn’t see “got it” – all I saw was the standard “SyntaxError: invalid syntax” traceback. I restarted N++ after commenting out the 2 lines in code.py.

I see what you mean now about overwriting the run_code function, but it seems like I could avoid that complexity and just copy+paste+modify the functionality of your “sys.excepthook = xxxxx_func” in startup.py and have that xxxxx_func function do what I want (e.g. console.show()) upon unhandled exception. Of course, currently since I don’t see the “got it” example working that is going to be problematic. :-)

Claudia Frank

@Alan-Kilborn

strange.

You do use python script 1.0.8?
On Windows or like me on linux?
Current npp version?

Cheers
Claudia

Claudia Frank

@Alan-Kilborn

wait - I guess I know why - just need to investigate why it didn’t happen to me.

Cheers
Claudia

Claudia Frank

Needs more investigation - will follow up on this tomorrow.

Good night
Cheers
Claudia

Alan Kilborn

@Claudia-Frank

Yes, using PS 1.0.8.0 on Win7. I renamed code.py (and deleted code.pyc) and restarted N++. I thought this would have really bad effects when trying to run PS’s in N++, but it had no effect at all (scripts ran just the same as always). This seems to indicate that code.py has no influence…but this is odd since you seem to see the effects of changing it. I’m confused…

Alan Kilborn

I’m hesitant to post this, because I really prefer the “hook” solution if it can be worked out, but I tried wrapping a “main” function:

def main():
    x=y  # cause exception as y is undefined

try:
    main()
except:
    console.show()

Running this results in a hard hang of Notepad++!

Claudia Frank

@Alan-Kilborn

ok - tried to understand the python script source code and this is what I assume is
what happens.

When executing python files, the one we create with Plugins->PythonScript-New Script
code.py is NOT used, instead the C++ implementation of the python interface,
namely PyRun_Simplefile. Makes a lot of sense.

When executing code in the console then code.py is used. (Not interesting for this issue)

Because of this, there is no need to change code.py but to make a global execption hook
working we have to put the following code into one of the startup.py files.
I prefer user startup.py but machine startup.py will work too.

import sys

def my_logging_func(exctype, value, traceback):
    console.show()
    console.write('{}\n{}\n{}\n'.format(exctype, value, traceback))

sys.excepthook = my_logging_func

Of course the my_logging_func code could look different for each.
But the parameters need to be 3!

When does it fail?
Python interpreter tries to compile the source before it gets executed and that means
if an exception is raised while compiling the source which includes the exception hook,
the hook cannot be installed.

Concerning the console.show() freeze, I’m using the console.show() since I started
with python script. I also tried your example it is working for me.

When your npp hangs than it means that python script created a deadlock.
But what could be the cause when running console.show()?
Can you run console.hide() when you open the console manually (via menu)?
Is there something special in your startup.pys?

Cheers
Claudia

Claudia Frank

@Alan-Kilborn

came just into my mind - could it be that you are using callbacks in your
startup.py files which could jump in?

Cheers
Claudia

Alan Kilborn

@Claudia-Frank

Okay, I disabled code initiated from startup.py that had a callback associated with it, and…everything (the “my_logging_func” stuff, and the “try/main()” stuff) discussed above now works. So the question becomes, what do the callbacks have to do with anything, as long as the callbacks don’t contain any code with unhandled exceptions?

And then the next question is, how do I get it all…my code with callbacks, and a custom exception handler (which does the console.show() )?

And again, Claudia, thank you for your diligence!

Claudia Frank

@Alan-Kilborn

Just a quick update - the exception hook should work together with your callbacks,
opening the console is the problem. If you want to know more about this there must
be an old thread at sourceforge forum.
To overcome this, use notepad.runPluginCommand(‘Python Script’, ‘Show Console’)
instead of console.show() (Please double check syntax)

Cheers
Claudia

Alan Kilborn

@Claudia-Frank

I found this at sourceforge. It doesn’t really detail anything, but it reminds me of our current discussion!
https://sourceforge.net/p/npppythonscript/discussion/1188885/thread/a1ec71f7/?limit=25#c261

Alan Kilborn

So here’s what I ended up embedding in startup.py…seems to do the job and meet the original requirement:

import traceback
def custom_exception_handler_func(exctype, value, trace_back):
    notepad.runPluginCommand('Python Script', 'Show Console')  # can't/don't use console.show()
    sys.stderr.write('(Single-level) Traceback:' + '\n')
    sys.stderr.write(traceback.format_tb(trace_back)[-1])  # only write out ONE level
sys.excepthook = custom_exception_handler_func

Claudia Frank

@Alan-Kilborn

I found this at sourceforge. It doesn’t really detail anything, but it reminds me of our current discussion!
https://sourceforge.net/p/npppythonscript/discussion/1188885/thread/a1ec71f7/?limit=25#c261

No, I have something in my mind related to the problems using console object
together with editor callbacks. Did a quick search but wasn’t able to find it.

Anyway, good to see that you have a working solution and I hope you don’t open
new python script related threads in the near future. Don’t get me wrong,
but it looks like have a knack to find all those nasty issues ;-)

Cheers
Claudia

Alan Kilborn

@Claudia-Frank

Again, many thanks for your support. And I have no more “toughies” in the queue for you…but…when I do I know where (and who) to go to! :-)

Also, my most recent example lacks an “import sys” to be complete…

DaveyD

@Claudia-Frank and @Alan-Kilborn,
Thanks for this thread - this is a nice feature that I’ve wanted for a while - now I have it! Thanks.

Regarding the console.show() crashing np++, I’ve had this issue before and it was also connected to a callback. See here and here.
At the time, Dave fixed it for us and it’s been working since.
Maybe those threads can give some clues…

Regards,
David

Claudia Frank

@DaveyD

David, thx - that is exactly the thread (second link) I was looking for.
Totally forgot (or didn’t even see) that there is an unofficial python script 1.1.1 .
But downloading and installing seems to confirm, the console object is working on
my side as well now.

Cheers
Claudia

Alan Kilborn

@Claudia-Frank

Is Pythonscript 1.1.1 something you are moving forward with for all of your PS development…or were you just doing a quick test in a sandbox install?

Claudia Frank

@Alan-Kilborn

was just a quick test - I stay with the official 1.0.8.

Cheers
Claudia

Alan Kilborn

Some additional time with the exception handler in place has shown the need for some tweaks to the code. Passing it along:

# set new exception hook so that when unhandled exception occurs while running a script,
#  it will force the Pythonscript console window to be shown
import sys, traceback
def custom_exception_handler_func(exctype, value, trace_back):
    notepad.runPluginCommand('Python Script', 'Show Console')  # can't/don't use console.show() in PS 1.0.8.0
    sys.stderr.write('(Single-level) Traceback:' + '\n')
    sys.stderr.write(traceback.format_tb(trace_back)[-1])  # only write out ONE level
    sys.stderr.write(str(exctype) + ':  ' + str(value) + '\n')
sys.excepthook = custom_exception_handler_func