Remove extra params from is_moving

src/viam/components/arm/client.py

@@ -93,9 +93,7 @@ async def stop(
         request = StopRequest(name=self.name, extra=dict_to_struct(extra))
         await self.client.Stop(request, timeout=timeout)
 
-    async def is_moving(self, *, extra: Optional[Dict[str, Any]] = None, timeout: Optional[float] = None) -> bool:
-        if extra is None:
-            extra = {}
+    async def is_moving(self, *, timeout: Optional[float] = None) -> bool:
         request = IsMovingRequest(name=self.name)
         response: IsMovingResponse = await self.client.IsMoving(request, timeout=timeout)
         return response.is_moving

src/viam/components/arm/service.py

@@ -1,6 +1,6 @@
 from grpclib.server import Stream
-from viam.errors import MethodNotImplementedError
 
+from viam.errors import MethodNotImplementedError
 from viam.proto.common import (
     DoCommandRequest,
     DoCommandResponse,

src/viam/components/gantry/client.py

@@ -62,9 +62,7 @@ async def stop(self, *, extra: Optional[Dict[str, Any]] = None, timeout: Optiona
         request = StopRequest(name=self.name, extra=dict_to_struct(extra))
         await self.client.Stop(request, timeout=timeout)
 
-    async def is_moving(self, *, extra: Optional[Dict[str, Any]] = None, timeout: Optional[float] = None) -> bool:
-        if extra is None:
-            extra = {}
+    async def is_moving(self, *, timeout: Optional[float] = None) -> bool:
         request = IsMovingRequest(name=self.name)
         response: IsMovingResponse = await self.client.IsMoving(request, timeout=timeout)
         return response.is_moving

src/viam/components/gripper/client.py

@@ -47,9 +47,7 @@ async def stop(self, *, extra: Optional[Dict[str, Any]] = None, timeout: Optiona
         request = StopRequest(name=self.name, extra=dict_to_struct(extra))
         await self.client.Stop(request, timeout=timeout)
 
-    async def is_moving(self, *, extra: Optional[Dict[str, Any]] = None, timeout: Optional[float] = None) -> bool:
-        if extra is None:
-            extra = {}
+    async def is_moving(self, *, timeout: Optional[float] = None) -> bool:
         request = IsMovingRequest(name=self.name)
         response: IsMovingResponse = await self.client.IsMoving(request, timeout=timeout)
         return response.is_moving

src/viam/components/motor/client.py

@@ -132,9 +132,7 @@ async def is_powered(
         response: IsPoweredResponse = await self.client.IsPowered(request, timeout=timeout)
         return response.is_on, response.power_pct
 
-    async def is_moving(self, *, extra: Optional[Dict[str, Any]] = None, timeout: Optional[float] = None) -> bool:
-        if extra is None:
-            extra = {}
+    async def is_moving(self, *, timeout: Optional[float] = None) -> bool:
         request = IsMovingRequest(name=self.name)
         response: IsMovingResponse = await self.client.IsMoving(request, timeout=timeout)
         return response.is_moving

src/viam/components/servo/client.py

@@ -1,4 +1,4 @@
-from typing import Any, Dict, Mapping, Optional
+from typing import Any, Mapping, Optional
 
 from grpclib.client import Channel
 
@@ -47,9 +47,7 @@ async def stop(self, *, extra: Optional[Mapping[str, Any]] = None, timeout: Opti
         request = StopRequest(name=self.name, extra=dict_to_struct(extra))
         await self.client.Stop(request, timeout=timeout)
 
-    async def is_moving(self, *, extra: Optional[Dict[str, Any]] = None, timeout: Optional[float] = None) -> bool:
-        if extra is None:
-            extra = {}
+    async def is_moving(self, *, timeout: Optional[float] = None) -> bool:
         request = IsMovingRequest(name=self.name)
         response: IsMovingResponse = await self.client.IsMoving(request, timeout=timeout)
         return response.is_moving

src/viam/resource/rpc_service_base.py

@@ -1,5 +1,5 @@
 import abc
-from typing import Type, TYPE_CHECKING
+from typing import TYPE_CHECKING, Type
 
 from viam.components.component_base import ComponentBase
 from viam.errors import ResourceNotFoundError

src/viam/robot/service.py

@@ -4,6 +4,8 @@
 from grpclib.server import Stream
 
 from viam import logging
+from viam.components.movement_sensor import MovementSensor
+from viam.components.sensor import Sensor
 from viam.errors import MethodNotImplementedError, ViamGRPCError
 from viam.proto.common import ResourceName
 from viam.proto.robot import (
@@ -43,8 +45,6 @@
 from viam.resource.registry import Registry
 from viam.resource.rpc_service_base import ResourceRPCServiceBase
 from viam.utils import resource_names_for_resource, struct_to_dict
-from viam.components.sensor import Sensor
-from viam.components.movement_sensor import MovementSensor
 
 LOGGER = logging.getLogger(__name__)