Opened 8 years ago

Closed 7 years ago

#1069 closed defect (fixed)

Trigger error messages difficult to debug

Reported by: Tristan Croll Owned by: Greg Couch
Priority: moderate Milestone:
Component: Core Version:
Keywords: Cc:
Blocked By: Blocking:
Notify when closed: Platform: all
Project: ChimeraX

Description

At present, if a callback raises an exception its _TriggerHandler catches it and prints to the log something along the lines of:

Error processing trigger "selection changed"

... which in a large and complex application is not all that helpful in tracking down the culprit. I understand the rationale for not wanting to fully raise the exception, but what about amending lines 127-128 of triggerset.py to the following?

        except Exception as e:
            _report('%s "%s": %s' % (TRIGGER_ERROR, self._name, str(e)))

In my case, that now gives me:

Error processing trigger "selection changed": 'Isolde' object has no attribute '_total_mobile'

... which gives me a clear indication of where to look.

Change History (1)

comment:1 by Greg Couch, 7 years ago

Resolution: fixed
Status: assignedclosed

Excellent suggestion, finally implemented.

Note: See TracTickets for help on using tickets.