Systematize setStagePosition signature, make None speed sentinel (#128)

* Systematize `setStagePosition` signature, make `None` a sentinel for speed

* Do not introduce `speed` to Jeol and Simu if they are not supported

* Apparently simu used to have unused speed for some reason.

Author: Baharis

src/instamatic/microscope/base.py

@@ -188,6 +188,8 @@ def setStagePosition(
         z: Optional[int_nm],
         a: Optional[float_deg],
         b: Optional[float_deg],
+        *,
+        speed: Optional[float],
         wait: bool,
     ) -> None:
         pass

src/instamatic/microscope/interface/fei_microscope.py

@@ -165,13 +165,16 @@ def setStagePosition(
         z: Optional[int_nm] = None,
         a: Optional[float_deg] = None,
         b: Optional[float_deg] = None,
+        *,
+        speed: Optional[float] = None,
         wait: bool = True,
-        speed: float = 1.0,
     ) -> None:
         """X, y, z in the system are in unit of meters, angles in radians."""
         pos = self.stage.Position
         axis = 0
 
+        if speed is None:
+            speed = 1.0
         if speed > 1 or speed < 0:
             raise FEIValueError(f'setStageSpeed value must be between 0 and 1. Input: {speed}')
 

src/instamatic/microscope/interface/fei_simu_microscope.py

@@ -252,8 +252,9 @@ def setStagePosition(
         z: Optional[int_nm] = None,
         a: Optional[float_deg] = None,
         b: Optional[float_deg] = None,
+        *,
+        speed: Optional[float] = None,
         wait: bool = True,
-        speed: float = 1.0,
     ) -> None:
         if z is not None:
             self.setStageZ(z)
@@ -273,6 +274,8 @@ def setStagePosition_nw(
         z: Optional[int_nm] = None,
         a: Optional[float_deg] = None,
         b: Optional[float_deg] = None,
+        *,
+        speed: Optional[float] = None,
         wait: bool = False,
     ) -> None:
         if z is not None:

src/instamatic/microscope/interface/jeol_microscope.py

@@ -341,6 +341,7 @@ def setStagePosition(
         z: Optional[int_nm] = None,
         a: Optional[float_deg] = None,
         b: Optional[float_deg] = None,
+        *,
         wait: bool = True,
     ) -> None:
         if z is not None:

src/instamatic/microscope/interface/simu_microscope.py

@@ -433,8 +433,9 @@ def setStagePosition(
         z: Optional[int_nm] = None,
         a: Optional[float_deg] = None,
         b: Optional[float_deg] = None,
-        speed: float = -1,
+        *,
         wait: bool = True,
+        speed: Optional[float] = None,
     ) -> None:
         if z is not None:
             self.setStageZ(z, wait=wait)