Log error on order conversion problems

Signed-off-by: cwasicki <[email protected]>

Author: cwasicki

src/frequenz/client/electricity_trading/_types.py

@@ -99,11 +99,18 @@ def from_pb(cls, price: price_pb2.Price) -> Self:
 
         Returns:
             Price object corresponding to the protobuf message.
+
+        Raises:
+            Exception: If an error occurs during conversion.
         """
-        return cls(
-            amount=Decimal(price.amount.value),
-            currency=Currency.from_pb(price.currency),
-        )
+        try:
+            return cls(
+                amount=Decimal(price.amount.value),
+                currency=Currency.from_pb(price.currency),
+            )
+        except Exception as e:
+            _logger.error("Error converting price `%s`: %s", price, e)
+            raise
 
     def to_pb(self) -> price_pb2.Price:
         """Convert a Price object to protobuf Price.
@@ -139,8 +146,15 @@ def from_pb(cls, power: power_pb2.Power) -> Self:
 
         Returns:
             Power object corresponding to the protobuf message.
+
+        Raises:
+            Exception: If an error occurs during conversion.
         """
-        return cls(mw=Decimal(power.mw.value))
+        try:
+            return cls(mw=Decimal(power.mw.value))
+        except Exception as e:
+            _logger.error("Error converting power `%s`: %s", power, e)
+            raise
 
     def to_pb(self) -> power_pb2.Power:
         """Convert a Power object to protobuf Power.
@@ -959,42 +973,51 @@ def from_pb(cls, order: electricity_trading_pb2.Order) -> Self:
 
         Returns:
             Order object corresponding to the protobuf message.
-        """
-        return cls(
-            delivery_area=DeliveryArea.from_pb(order.delivery_area),
-            delivery_period=DeliveryPeriod.from_pb(order.delivery_period),
-            type=OrderType.from_pb(order.type),
-            side=MarketSide.from_pb(order.side),
-            price=Price.from_pb(order.price),
-            quantity=Power.from_pb(order.quantity),
-            stop_price=(
-                Price.from_pb(order.stop_price)
-                if order.HasField("stop_price")
-                else None
-            ),
-            peak_price_delta=(
-                Price.from_pb(order.peak_price_delta)
-                if order.HasField("peak_price_delta")
-                else None
-            ),
-            display_quantity=(
-                Power.from_pb(order.display_quantity)
-                if order.HasField("display_quantity")
-                else None
-            ),
-            execution_option=(
-                OrderExecutionOption.from_pb(order.execution_option)
-                if order.HasField("execution_option")
-                else None
-            ),
-            valid_until=(
-                order.valid_until.ToDatetime(tzinfo=timezone.utc)
-                if order.HasField("valid_until")
-                else None
-            ),
-            payload=json_format.MessageToDict(order.payload) if order.payload else None,
-            tag=order.tag if order.tag else None,
-        )
+
+        Raises:
+            Exception: If an error occurs during conversion.
+        """
+        try:
+            return cls(
+                delivery_area=DeliveryArea.from_pb(order.delivery_area),
+                delivery_period=DeliveryPeriod.from_pb(order.delivery_period),
+                type=OrderType.from_pb(order.type),
+                side=MarketSide.from_pb(order.side),
+                price=Price.from_pb(order.price),
+                quantity=Power.from_pb(order.quantity),
+                stop_price=(
+                    Price.from_pb(order.stop_price)
+                    if order.HasField("stop_price")
+                    else None
+                ),
+                peak_price_delta=(
+                    Price.from_pb(order.peak_price_delta)
+                    if order.HasField("peak_price_delta")
+                    else None
+                ),
+                display_quantity=(
+                    Power.from_pb(order.display_quantity)
+                    if order.HasField("display_quantity")
+                    else None
+                ),
+                execution_option=(
+                    OrderExecutionOption.from_pb(order.execution_option)
+                    if order.HasField("execution_option")
+                    else None
+                ),
+                valid_until=(
+                    order.valid_until.ToDatetime(tzinfo=timezone.utc)
+                    if order.HasField("valid_until")
+                    else None
+                ),
+                payload=(
+                    json_format.MessageToDict(order.payload) if order.payload else None
+                ),
+                tag=order.tag if order.tag else None,
+            )
+        except Exception as e:
+            _logger.error("Error converting order `%s`: %s", order, e)
+            raise
 
     def to_pb(self) -> electricity_trading_pb2.Order:
         """