return value_pb for less unnecessary conversions

gaddison-bdai · gaddison-bdai · commit b1a226675212 · 2025-03-12T17:13:22.000-04:00
diff --git a/spot_choreo_utils/spot_choreo_utils/choreo_creation/choreo_builders/sequence_builder.py b/spot_choreo_utils/spot_choreo_utils/choreo_creation/choreo_builders/sequence_builder.py
@@ -170,7 +170,7 @@ def _clamp_param(self, name: str, value_pb: DoubleValue) -> DoubleValue:
                 )
             return DoubleValue(value=bounds[1])
         else:
-            return DoubleValue(value=val)
+            return value_pb
 
     def validate_rotate_body(self, params: RotateBodyParams) -> bool:
         params.start_slice.CopyFrom(self._clamp_param("start_slice", params.start_slice))

Original file line number	Diff line number	Diff line change
`@@ -170,7 +170,7 @@ def _clamp_param(self, name: str, value_pb: DoubleValue) -> DoubleValue:`
`170`	`170`	`)`
`171`	`171`	`return DoubleValue(value=bounds[1])`
`172`	`172`	`else:`
`173`		`- return DoubleValue(value=val)`
	`173`	`+ return value_pb`
`174`	`174`
`175`	`175`	`def validate_rotate_body(self, params: RotateBodyParams) -> bool:`
`176`	`176`	`params.start_slice.CopyFrom(self._clamp_param("start_slice", params.start_slice))`