[Dextrose] [Design]: SL Bug #2063 (Sugar should bring up an alert when an unhandled Python exception occurs)

[Mukul Gupta]

Team,

Wish to have your feedback on the design aspect of the ticket 2063 (Sugar
should bring up an alert when an unhandled Python exception occurs) - please
find the git diff attached below. This is an attempt to display an alert
when an unhandled Python exception occurs in the journal. Unfortunately,
when the exception takes place in journalactivity.py , I am neither unable
to notice the Journal Icon (it disappears),



diff --git a/journalactivity.py b/journalactivity.py
index 44cc018..36a2e2e 100644
--- a/journalactivity.py
+++ b/journalactivity.py
@@ -358,8 +358,20 @@ class JournalActivity(Window):
         self.show_main_view()
         self.search_grab_focus()

+    def uncaught_exception_alert(self):
+        alert = ErrorAlert(title="Operation could not be performed",
msg="Please check the logviewer activity for details ")
+    alert.connect('response', self.__alert_response_cb)
+    self.add_alert(alert)
+    alert.show()
+
 _journal = None

+def _alert_excepthook(exc_type, exc_value, traceback):
+    logging.exception('Unhandled Python exception: %s', repr((exc_type,
exc_value, traceback)))
+    _journal.uncaught_exception_alert()
+
+sys.excepthook = _alert_excepthook
+
 def get_journal():
     global _journal
     if _journal is None:
-- 
1.7.0.4


I wish to request you to please review the code and suggest desired changes,
if any.

Moreover, I have two important questions regarding the bug.

1. To catch all unhandled python exceptions in sugar, where exactly should
we be looking forward to be the venue for adding the functionality?

    Adding it in journalactivity.py doesn't seem to serve all purposes. It
has to be added somewhere which is being used all the time.

2. Wish if you could recommend on the GUI feature that could be used for
displaying the alert message to the user.

Looking forward to for your valuable suggestions,


Regards,

Mukul Gupta
Research Engineer,SEETA


On Tue, Oct 12, 2010 at 12:45 AM, Mukul Gupta <mukul at seeta.in> wrote:

> Team,
>
> I had a discussion with Aleksey this morning at IRC on the feedback
> recommended yesterday. Thank you Daniel and Tomeu. Appreciate your ideas and
> pointers.
>
> We brainstormed on how the implementation of this functionality could be
> done, and found that whenever any uncaught exception is raised, the
> interpreter calls the sys.excepthook(type,value,traceback).
>
> Wish to have the feedback on whether excepthook implementation would be
> helpful to our use-case both short-term and long-term. Also, it would be
> wonderful if feedback could be shared on its implementation in the context
> of Sugar, or otherwise.
>
> Looking forward to hearing from you.
>
> Regards,
>
> Mukul Gupta
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/dextrose/attachments/20101014/c6f7f9b6/attachment.html>