(fix) Changed gas cost of spot orders in general, to solve out of gas…

… issues when using the estimator
InjectiveLabs · Jan 24, 2024 · e315110 · e315110
1 parent 53496cf
commit e315110
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/pyinjective/core/gas_limit_estimator.py b/pyinjective/core/gas_limit_estimator.py
@@ -12,13 +12,12 @@
     tx_pb2 as injective_exchange_tx_pb,
 )
 
-SPOT_ORDER_CREATION_GAS_LIMIT = 45_000
+SPOT_ORDER_CREATION_GAS_LIMIT = 50_000
 DERIVATIVE_ORDER_CREATION_GAS_LIMIT = 70_000
 SPOT_ORDER_CANCELATION_GAS_LIMIT = 50_000
 DERIVATIVE_ORDER_CANCELATION_GAS_LIMIT = 60_000
-# POST ONLY orders take around 60% (for spot) and 50% (for derivative) more gas to create than normal orders
-# due to the required validations
-SPOT_POST_ONLY_ORDER_MULTIPLIER = 0.6
+# POST ONLY orders take around 50% more gas to create than normal orders due to the required validations
+SPOT_POST_ONLY_ORDER_MULTIPLIER = 0.5
 DERIVATIVE_POST_ONLY_ORDER_MULTIPLIER = 0.5
 
 

diff --git a/tests/core/test_gas_limit_estimator.py b/tests/core/test_gas_limit_estimator.py
@@ -46,7 +46,7 @@ def test_estimation_for_batch_create_spot_limit_orders(self):
         message = composer.MsgBatchCreateSpotLimitOrders(sender="sender", orders=orders)
         estimator = GasLimitEstimator.for_message(message=message)
 
-        expected_order_gas_limit = 45000
+        expected_order_gas_limit = 50000
         expected_message_gas_limit = 15000
 
         assert (expected_order_gas_limit * 2) + expected_message_gas_limit == estimator.gas_limit()
@@ -172,7 +172,7 @@ def test_estimation_for_batch_update_orders_to_create_spot_orders(self):
         )
         estimator = GasLimitEstimator.for_message(message=message)
 
-        expected_order_gas_limit = 45_000
+        expected_order_gas_limit = 50_000
         expected_message_gas_limit = 15_000
 
         assert (expected_order_gas_limit * 2) + expected_message_gas_limit == estimator.gas_limit()
@@ -462,7 +462,7 @@ def test_estimation_for_exec_message(self):
 
         estimator = GasLimitEstimator.for_message(message=message)
 
-        expected_order_gas_limit = 45_000
+        expected_order_gas_limit = 50_000
         expected_inner_message_gas_limit = 15_000
         expected_exec_message_gas_limit = 8_000