Remove unnecessary sleep and add performance TODO (#1158)

* Remove unnecessary sleep and add performance TODO

Removed a redundant time.sleep call in SyntheticStage's move_absolute method to improve performance. Added a TODO comment in Microscope to highlight the cost of repeated dictionary lookups for image_mode during z-stack acquisition, suggesting future optimization.

* Update synthetic.py

Author: kevin-m-dean

src/navigate/model/devices/stage/synthetic.py

@@ -125,9 +125,6 @@ def move_axis_absolute(
         if axis_abs == -1e50:
             return False
 
-        if wait_until_done:
-            time.sleep(0.025)
-
         # Move the stage
         setattr(self, f"{axis}_pos", axis_abs)
         return True
@@ -157,8 +154,6 @@ def move_absolute(
 
         for axis in abs_pos_dict:
             setattr(self, f"{axis}_pos", abs_pos_dict[axis])
-        if wait_until_done is True:
-            time.sleep(0.025)
 
         return True
 

src/navigate/model/microscope.py

@@ -892,6 +892,11 @@ def move_stage(
         success : bool
             True if stage is successfully moved, False otherwise.
         """
+
+        # TODO: Calling the proxy dictionary costs ~3 ms, and is performed for every step
+        # in the z-stack. To optimize performance, we should retrieve the image mode
+        # once and avoid repeated dictionary lookups.
+
         if self.configuration["experiment"]["MicroscopeState"]["image_mode"] in (
             "z-stack",
             "customized",