(fix) Changes to gas limit estimation to reduce a little bit gas consumption

Author: abel

pyinjective/async_client.py

@@ -302,7 +302,7 @@ async def get_account(self, address: str) -> Optional[account_pb2.EthAccount]:
         try:
             metadata = await self.load_cookie(type="chain")
             account_any = (await self.stubAuth.Account(
-                auth_query.QueryAccountRequest.__call__(address=address), metadata=metadata
+                auth_query.QueryAccountRequest(address=address), metadata=metadata
             )).account
             account = account_pb2.EthAccount()
             if account_any.Is(account.DESCRIPTOR):
@@ -339,7 +339,7 @@ async def simulate_tx(
         try:
             req = tx_service.SimulateRequest(tx_bytes=tx_byte)
             metadata = await self.load_cookie(type="chain")
-            return await self.stubTx.Simulate.__call__(req, metadata=metadata), True
+            return await self.stubTx.Simulate(request=req, metadata=metadata), True
         except grpc.RpcError as err:
             return err, False
 
@@ -348,23 +348,23 @@ async def send_tx_sync_mode(self, tx_byte: bytes) -> abci_type.TxResponse:
             tx_bytes=tx_byte, mode=tx_service.BroadcastMode.BROADCAST_MODE_SYNC
         )
         metadata = await self.load_cookie(type="chain")
-        result = await self.stubTx.BroadcastTx.__call__(req, metadata=metadata)
+        result = await self.stubTx.BroadcastTx(request=req, metadata=metadata)
         return result.tx_response
 
     async def send_tx_async_mode(self, tx_byte: bytes) -> abci_type.TxResponse:
         req = tx_service.BroadcastTxRequest(
             tx_bytes=tx_byte, mode=tx_service.BroadcastMode.BROADCAST_MODE_ASYNC
         )
         metadata = await self.load_cookie(type="chain")
-        result = await self.stubTx.BroadcastTx.__call__(req, metadata=metadata)
+        result = await self.stubTx.BroadcastTx(request=req, metadata=metadata)
         return result.tx_response
 
     async def send_tx_block_mode(self, tx_byte: bytes) -> abci_type.TxResponse:
         req = tx_service.BroadcastTxRequest(
             tx_bytes=tx_byte, mode=tx_service.BroadcastMode.BROADCAST_MODE_BLOCK
         )
         metadata = await self.load_cookie(type="chain")
-        result = await self.stubTx.BroadcastTx.__call__(req, metadata=metadata)
+        result = await self.stubTx.BroadcastTx(request=req, metadata=metadata)
         return result.tx_response
 
     async def get_chain_id(self) -> str:
@@ -632,7 +632,7 @@ async def stream_spot_markets(self, **kwargs):
             market_ids=kwargs.get("market_ids")
         )
         metadata = await self.load_cookie(type="exchange")
-        return self.stubSpotExchange.StreamMarkets.__call__(req, metadata=metadata)
+        return self.stubSpotExchange.StreamMarkets(request=req, metadata=metadata)
 
     async def get_spot_orderbookV2(self, market_id: str):
         req = spot_exchange_rpc_pb.OrderbookV2Request(market_id=market_id)
@@ -689,12 +689,12 @@ async def get_spot_trades(self, **kwargs):
     async def stream_spot_orderbook_snapshot(self, market_ids: List[str]):
         req = spot_exchange_rpc_pb.StreamOrderbookV2Request(market_ids=market_ids)
         metadata = await self.load_cookie(type="exchange")
-        return self.stubSpotExchange.StreamOrderbookV2.__call__(req, metadata=metadata)
+        return self.stubSpotExchange.StreamOrderbookV2(request=req, metadata=metadata)
 
     async def stream_spot_orderbook_update(self, market_ids: List[str]):
         req = spot_exchange_rpc_pb.StreamOrderbookUpdateRequest(market_ids=market_ids)
         metadata = await self.load_cookie(type="exchange")
-        return self.stubSpotExchange.StreamOrderbookUpdate.__call__(req, metadata=metadata)
+        return self.stubSpotExchange.StreamOrderbookUpdate(request=req, metadata=metadata)
 
     async def stream_spot_orders(self, market_id: str, **kwargs):
         req = spot_exchange_rpc_pb.StreamOrdersRequest(
@@ -703,7 +703,7 @@ async def stream_spot_orders(self, market_id: str, **kwargs):
             subaccount_id=kwargs.get("subaccount_id"),
         )
         metadata = await self.load_cookie(type="exchange")
-        return self.stubSpotExchange.StreamOrders.__call__(req, metadata=metadata)
+        return self.stubSpotExchange.StreamOrders(request=req, metadata=metadata)
 
     async def stream_historical_spot_orders(self, market_id: str, **kwargs):
         req = spot_exchange_rpc_pb.StreamOrdersHistoryRequest(
@@ -715,7 +715,7 @@ async def stream_historical_spot_orders(self, market_id: str, **kwargs):
             execution_types=kwargs.get("execution_types")
         )
         metadata = await self.load_cookie(type="exchange")
-        return self.stubSpotExchange.StreamOrdersHistory.__call__(req, metadata=metadata)
+        return self.stubSpotExchange.StreamOrdersHistory(request=req, metadata=metadata)
 
     async def stream_historical_derivative_orders(self, market_id: str, **kwargs):
         req = derivative_exchange_rpc_pb.StreamOrdersHistoryRequest(
@@ -727,7 +727,7 @@ async def stream_historical_derivative_orders(self, market_id: str, **kwargs):
             execution_types=kwargs.get("execution_types")
         )
         metadata = await self.load_cookie(type="exchange")
-        return self.stubDerivativeExchange.StreamOrdersHistory.__call__(req, metadata=metadata)
+        return self.stubDerivativeExchange.StreamOrdersHistory(request=req, metadata=metadata)
 
     async def stream_spot_trades(self, **kwargs):
         req = spot_exchange_rpc_pb.StreamTradesRequest(
@@ -740,7 +740,7 @@ async def stream_spot_trades(self, **kwargs):
             execution_types=kwargs.get("execution_types"),
         )
         metadata = await self.load_cookie(type="exchange")
-        return self.stubSpotExchange.StreamTrades.__call__(req, metadata=metadata)
+        return self.stubSpotExchange.StreamTrades(request=req, metadata=metadata)
 
     async def get_spot_subaccount_orders(self, subaccount_id: str, **kwargs):
         req = spot_exchange_rpc_pb.SubaccountOrdersListRequest(
@@ -780,7 +780,7 @@ async def stream_derivative_markets(self, **kwargs):
             market_ids=kwargs.get("market_ids")
         )
         metadata = await self.load_cookie(type="exchange")
-        return self.stubDerivativeExchange.StreamMarket.__call__(req, metadata=metadata)
+        return self.stubDerivativeExchange.StreamMarket(request=req, metadata=metadata)
 
     async def get_derivative_orderbook(self, market_id: str):
         req = derivative_exchange_rpc_pb.OrderbookV2Request(market_id=market_id)
@@ -846,16 +846,12 @@ async def get_derivative_trades(self, **kwargs):
     async def stream_derivative_orderbook_snapshot(self, market_ids: List[str]):
         req = derivative_exchange_rpc_pb.StreamOrderbookV2Request(market_ids=market_ids)
         metadata = await self.load_cookie(type="exchange")
-        return self.stubDerivativeExchange.StreamOrderbookV2.__call__(
-            req, metadata=metadata
-        )
+        return self.stubDerivativeExchange.StreamOrderbookV2(request=req, metadata=metadata)
 
     async def stream_derivative_orderbook_update(self, market_ids: List[str]):
         req = derivative_exchange_rpc_pb.StreamOrderbookUpdateRequest(market_ids=market_ids)
         metadata = await self.load_cookie(type="exchange")
-        return self.stubDerivativeExchange.StreamOrderbookUpdate.__call__(
-            req, metadata=metadata
-        )
+        return self.stubDerivativeExchange.StreamOrderbookUpdate(request=req, metadata=metadata)
 
     async def stream_derivative_orders(self, market_id: str, **kwargs):
         req = derivative_exchange_rpc_pb.StreamOrdersRequest(
@@ -864,7 +860,7 @@ async def stream_derivative_orders(self, market_id: str, **kwargs):
             subaccount_id=kwargs.get("subaccount_id"),
         )
         metadata = await self.load_cookie(type="exchange")
-        return self.stubDerivativeExchange.StreamOrders.__call__(req, metadata=metadata)
+        return self.stubDerivativeExchange.StreamOrders(request=req, metadata=metadata)
 
     async def stream_derivative_trades(self, **kwargs):
         req = derivative_exchange_rpc_pb.StreamTradesRequest(
@@ -879,7 +875,7 @@ async def stream_derivative_trades(self, **kwargs):
             execution_types=kwargs.get("execution_types"),
         )
         metadata = await self.load_cookie(type="exchange")
-        return self.stubDerivativeExchange.StreamTrades.__call__(req, metadata=metadata)
+        return self.stubDerivativeExchange.StreamTrades(request=req, metadata=metadata)
 
     async def get_derivative_positions(self, **kwargs):
         req = derivative_exchange_rpc_pb.PositionsRequest(
@@ -901,9 +897,7 @@ async def stream_derivative_positions(self, **kwargs):
             subaccount_ids=kwargs.get("subaccount_ids"),
         )
         metadata = await self.load_cookie(type="exchange")
-        return self.stubDerivativeExchange.StreamPositions.__call__(
-            req, metadata=metadata
-        )
+        return self.stubDerivativeExchange.StreamPositions(request=req, metadata=metadata)
 
     async def get_derivative_liquidable_positions(self, **kwargs):
         req = derivative_exchange_rpc_pb.LiquidablePositionsRequest(
@@ -979,4 +973,4 @@ async def stream_account_portfolio(self, account_address: str, **kwargs):
             type=kwargs.get("type")
         )
         metadata = await self.load_cookie(type="exchange")
-        return self.stubPortfolio.StreamAccountPortfolio.__call__(req, metadata=metadata)
+        return self.stubPortfolio.StreamAccountPortfolio(request=req, metadata=metadata)

pyinjective/core/broadcaster.py

@@ -287,8 +287,8 @@ async def configure_gas_fee_for_transaction(
 
 
 class MessageBasedTransactionFeeCalculator(TransactionFeeCalculator):
-    DEFAULT_GAS_LIMIT = 400_000
-    DEFAULT_EXCHANGE_GAS_LIMIT = 200_000
+    DEFAULT_GAS_LIMIT = 150_000
+    DEFAULT_EXCHANGE_GAS_LIMIT = 100_000
 
     def __init__(
             self,
@@ -313,11 +313,11 @@ def __init__(
                 Decimal("2.5") * self._base_gas_limit
             ),
             (
-                lambda message: "wasm" in self._message_type(message=message),
+                lambda message: "wasm." in self._message_type(message=message),
                 Decimal("1.5") * self._base_gas_limit
             ),
             (
-                lambda message: "exchange" in self._message_type(message=message),
+                lambda message: "exchange." in self._message_type(message=message),
                 Decimal("1") * self._base_exchange_gas_limit
             ),
             (
@@ -353,7 +353,7 @@ def _message_type(self, message: any_pb2.Any) -> str:
 
     def _is_governance_message(self, message: any_pb2.Any) -> bool:
         message_type = self._message_type(message=message)
-        return "gov" in message_type and ("MsgDeposit" in message_type or "MsgSubmitProposal" in message_type)
+        return "gov." in message_type and ("MsgDeposit" in message_type or "MsgSubmitProposal" in message_type)
 
     def _calculate_gas_limit(self, messages: List[any_pb2.Any]) -> int:
         total_gas_limit = Decimal("0")