Renamed GNSS.pause() to paused() to better convey temporary state wit…

…h contextmanager. Removed comment for self explaining action.
SeaTechRC · Sep 11, 2024 · 44a2d4f · 44a2d4f
1 parent 555555b
commit 44a2d4f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/field_friend/automations/navigation/navigation.py b/field_friend/automations/navigation/navigation.py
@@ -55,7 +55,7 @@ async def start(self) -> None:
                     await self._drive(self.DEFAULT_DRIVE_DISTANCE)
                 else:
                     await self._drive(distance)
-                    with self.gnss.pause(): # Pause GNSS for better local accuracy
+                    with self.gnss.paused(): # Pause GNSS for better local accuracy
                         await self.implement.start_workflow()
                         await self.implement.stop_workflow()
         except WorkflowException as e:

diff --git a/field_friend/localization/gnss.py b/field_friend/localization/gnss.py
@@ -95,7 +95,7 @@ async def check_gnss(self) -> None:
             self.current = None
 
     @contextlib.contextmanager
-    def pause(self):
+    def paused(self):
         try:
             self._is_paused = True
             yield
@@ -129,7 +129,6 @@ def _on_rtk_fix(self) -> None:
         if len(self.observed_poses) > self.needed_poses: # Ensure fresh data
             self.observed_poses.pop(0)
 
-        # Only update if unpaused
         if not self._is_paused:
             self._update_robot_pose()