Merge pull request #890 from annie-xd-wang/deal-with-removed/broken-plugins

Deal with removed/broken plugins

Author: AdvancedImagingUTSW

docs/source/user_guide/hardware/configurations/configuration_ctaslmv2.yaml

@@ -21,7 +21,7 @@ microscopes:
         serial_number: 302153
       flip_x: True
       flip_y: False
-      defect_correct_mode: 1.0
+      defect_correct_mode: 2.0
       delay: 20
     remote_focus_device:
       hardware:

docs/source/user_guide/hardware/configurations/configuration_mesospimbt.yaml

@@ -19,7 +19,7 @@ microscopes:
       hardware:
         type: HamamatsuOrca
         serial_number: 003209
-      defect_correct_mode: 1.0 #2.0
+      defect_correct_mode: 2.0 #2.0
       delay: 10 #ms
       settle_down: 0.0 #ms
       flip_x: True

docs/source/user_guide/hardware/remote_focus.rst

@@ -65,6 +65,8 @@ The `BLINK <https://www.thorlabs.com/thorproduct.cfm?partnumber=BLINK>`_ and the
 `Optotune Focus Tunable Lens <https://www.optotune.com/tunable-lenses>`_ are
 controlled with an analog signal from the DAQ.
 
+-----------------
+
 Thorlabs BLINK
 ~~~~~~~~~~~~~~
 The BLINK is a voice coil that is
@@ -89,6 +91,8 @@ voltage outside of its operating range.
 
 |
 
+-----------------
+
 Optotune Focus Tunable Lens
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~
 

docs/source/user_guide/hardware/stage.rst

@@ -95,6 +95,7 @@ first establish communication with the device using `ASI provided software <http
               flip_z: False
               flip_f: False
 
+-----------------
 
 MFC2000
 ~~~~~~~
@@ -146,6 +147,7 @@ MFC2000
 
 |
 
+-----------------
 
 MS2000
 ~~~~~~~
@@ -264,6 +266,8 @@ the software is restarted, it should work.
 
 |
 
+-----------------
+
 Physik Instrumente
 ------------------
 
@@ -292,6 +296,8 @@ or on a label on the side of your stage.
     repeatability of 2 microns, and a minimum incremental motion of 100 nm.
     This is potentially too coarse.
 
+-----------------
+
 C-884
 ~~~~~
 
@@ -341,6 +347,8 @@ C-884
 
 |
 
+-----------------
+
 E-709
 ~~~~~
 
@@ -449,6 +457,7 @@ positioning.
 
 |
 
+-----------------
 
 KST101
 ~~~~~~

src/navigate/controller/sub_controllers/plugins_controller.py

@@ -35,6 +35,7 @@
 import os
 import inspect
 import tkinter as tk
+from tkinter import messagebox
 
 # Third-party imports
 
@@ -135,16 +136,21 @@ def load_plugins(self):
                     )
                     if plugin_frame_module is None:
                         print(
-                            "Make sure that the plug in frame name is correct! "
-                            f"Plug in {plugin_name} needs to be reinstalled!"
+                            f"Make sure that the plugin frame name {plugin_class_name} is correct! "
+                            f"Plugin {plugin_name} needs to be uninstalled from navigate or reinstalled!"
                         )
-                        return
+                        continue
                     plugin_frame = getattr(
                         plugin_frame_module, f"{plugin_class_name}Frame"
                     )
                     plugin_controller_module = load_module_from_file(
                         f"{plugin_class_name}Controller", controller_file
                     )
+                    if plugin_controller_module is None:
+                        print(
+                            f"Make sure that the plugin controller {plugin_class_name} is correct! "
+                            f"Plugin {plugin_name} needs to be uninstalled from navigate or reinstalled!"
+                        )
                     plugin_controller = getattr(
                         plugin_controller_module, f"{plugin_class_name}Controller"
                     )
@@ -208,13 +214,22 @@ def build_tab_window(self, plugin_name, frame, controller):
         controller: object
             navigate controller
         """
-        plugin_frame = frame(self.view.settings)
-        self.view.settings.add(plugin_frame, text=plugin_name, sticky=tk.NSEW)
-        plugin_controller = controller(plugin_frame, self.parent_controller)
-        controller_name = (
-            "__plugin" + "_".join(plugin_name.lower().split()) + "_controller"
-        )
-        self.plugins_dict[controller_name] = plugin_controller
+        try:
+            plugin_frame = frame(self.view.settings)
+            self.view.settings.add(plugin_frame, text=plugin_name, sticky=tk.NSEW)
+            plugin_controller = controller(plugin_frame, self.parent_controller)
+            controller_name = (
+                "__plugin" + "_".join(plugin_name.lower().split()) + "_controller"
+            )
+            self.plugins_dict[controller_name] = plugin_controller
+        except:
+            messagebox.showwarning(
+                title="Warning",
+                message=(
+                    f"Plugin {plugin_name} has something went wrong."
+                    f"Please make sure the plugin works correctly or uninstall it!"
+                )
+            )
 
     def build_popup_window(self, plugin_name, frame, controller):
         """Build popup window for a plugin
@@ -264,7 +279,19 @@ def func(*args, **kwargs):
                 ),
             )
 
-        return func
+        def func_with_wrapper(*args, **kwargs):
+            try:
+                func(*args, **kwargs)
+            except:
+                messagebox.showwarning(
+                    title="Warning",
+                    message=(
+                        f"Plugin {plugin_name} has something went wrong."
+                        f"Please make sure the plugin works correctly or uninstall it from navigate!"
+                    )
+                )
+
+        return func_with_wrapper
 
 
 class UninstallPluginController(GUIController):
@@ -342,7 +369,7 @@ def uninstall_plugins(self, *args):
             save_yaml_file(
                 self.plugin_config_path, self.plugin_config, "plugins_config.yml"
             )
-            tk.messagebox.showwarning(
+            messagebox.showwarning(
                 title="Navigate",
                 message="Plugins are uninstalled! Please restart Navigate!",
             )