labscript-suite-temp-2 / blacs

BLACS, part of the labscript suite, provides an interface to hardware used to control a buffered experiment. It manages a queue of shots to be run as well as providing manual control over devices between shots.
Other
0 stars 0 forks source link

Add information on loaded plugins #50

Open philipstarkey opened 6 years ago

philipstarkey commented 6 years ago

Original report (archived issue) by Lars Kohfahl (Bitbucket: lkohfahl, GitHub: lakodarmstadt).


When implementing the latest progress_bar changes I stumpled over some problems with our own progress bar because I had the naming in the labconfig file wrong. However this was not noticable as labscript just silently included the new progress bar and did not inform me about the plugins that could not be loaded. I tried to write some lines to face this problem, which probably need some improvements before starting a PR.

[https://bitbucket-archive-test.labscriptsuite.org/#!/lkohfahl/blacs/commits/1c7d683103d6dea218de4eedd25020f7c78b5544 (labscript-suite-temp-2-archive/lkohfahl-blacs--forked-from--labscript_suite-blacs@27ece1936e8a284465590021bff5c68dcaec1fcd)](Link URL)

philipstarkey commented 6 years ago

Original comment by Philip Starkey (Bitbucket: philipstarkey, GitHub: philipstarkey).


Good idea! Generating a notification in the GUI would be good, as well as logging to the BLACS log file. Unsure if you can generate Notifications outside of a plugin... there's probably a way to do it though!

philipstarkey commented 6 years ago

Original comment by Chris Billington (Bitbucket: cbillington, GitHub: chrisjbillington).


Failed plugin instantiation I think is in the log file already. I think I recently added a change to make it a full traceback, it used to be a bit silent.

We're saying promote this failure to a notification?