Add gas estimation to kandel populate tx (#1881)

* Add gas estimation to kandel populate tx * Add block gas limit as upper range
mangrovedao · Apr 16, 2024 · f25ee0e · f25ee0e
1 parent d34b0db
commit f25ee0e
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,7 @@
 # Next version
 
+- Change gas estimation for kandel populate transaction.
+
 # 2.0.5-49
 
 - Add 20% to gas limit from the gas estimation

diff --git a/src/kandel/geometricKandel/geometricKandelInstance.ts b/src/kandel/geometricKandel/geometricKandelInstance.ts
@@ -362,6 +362,29 @@ class GeometricKandelInstance extends CoreKandelInstance {
         params.distribution.askGives,
       );
 
+      if (!overridesWithFunds.gasLimit) {
+        const [gasEstimation, block] = await Promise.all([
+          this.geometricKandel.estimateGas.populateFromOffset(
+            firstDistribution.from,
+            firstDistribution.to,
+            params.distribution.baseQuoteTickIndex0,
+            newBaseQuoteTickOffset,
+            params.distribution.firstAskIndex,
+            rawBidGives,
+            rawAskGives,
+            rawParameters,
+            rawDepositBaseAmount,
+            rawDepositQuoteAmount,
+            overridesWithFunds,
+          ),
+          this.geometricKandel.provider.getBlock("latest"),
+        ]);
+        overridesWithFunds.gasLimit = gasEstimation.mul(120).div(100);
+        if (overridesWithFunds.gasLimit.gt(block.gasLimit)) {
+          overridesWithFunds.gasLimit = block.gasLimit;
+        }
+      }
+
       const txs = [
         await this.geometricKandel.populateFromOffset(
           firstDistribution.from,