Skip to content

Conversation

@donosonaumczuk
Copy link
Member

No description provided.

@github-actions
Copy link

Changes to gas cost

Generated at commit: 405065b22601f445ffa8decb90bd30a7808a0d3d, compared to commit: b128d4e98a5287930428e46574cef252dfdaeab4

🧾 Summary (20% most significant diffs)

Contract Method Avg (+/-) %
ATokenVault deposit
initialize
maxDeposit
mint
redeem
withdraw
+2,148 ❌
+1,763 ❌
+92 ❌
+2,246 ❌
+666 ❌
+706 ❌
+1.39%
+0.47%
+0.82%
+2.02%
+0.62%
+0.76%

Full diff report 👇
Contract Deployment Cost (+/-) Method Min (+/-) % Avg (+/-) % Median (+/-) % Max (+/-) % # Calls (+/-)
ATokenVault 4,799,278 (0) convertToAssets
convertToShares
deposit
depositWithSig
initialize
maxDeposit
maxMint
maxRedeem
maxWithdraw
mint
mintWithSig
previewDeposit
previewMint
previewRedeem
previewWithdraw
redeem
redeemWithATokensWithSig
redeemWithSig
totalAssets
withdraw
withdrawATokensWithSig
withdrawWithSig
11,709 (0)
2,590 (0)
5,288 (0)
27,976 (0)
3,730 (0)
11,144 (+92)
13,364 (+92)
11,205 (+92)
11,139 (+92)
7,532 (0)
27,912 (0)
22,561 (+92)
22,586 (+92)
27,917 (+92)
18,811 (+92)
10,990 (0)
27,934 (0)
27,934 (0)
9,052 (0)
11,035 (0)
27,912 (0)
27,955 (0)
0.00%
0.00%
0.00%
0.00%
0.00%
+0.83%
+0.69%
+0.83%
+0.83%
0.00%
0.00%
+0.41%
+0.41%
+0.33%
+0.49%
0.00%
0.00%
0.00%
0.00%
0.00%
0.00%
0.00%
21,972 (-12)
12,193 (-30)
156,923 (+2,148)
51,910 (+240)
379,100 (+1,763)
11,322 (+92)
22,663 (+92)
30,309 (+92)
50,552 (+44)
113,322 (+2,246)
51,900 (+240)
23,658 (+10)
22,586 (+92)
28,250 (+91)
27,868 (+56)
107,494 (+666)
48,553 (+19)
51,561 (+204)
10,828 (-12)
93,274 (+706)
48,785 (+19)
51,836 (+204)
-0.05%
-0.25%
+1.39%
+0.46%
+0.47%
+0.82%
+0.41%
+0.30%
+0.09%
+2.02%
+0.46%
+0.04%
+0.41%
+0.32%
+0.20%
+0.62%
+0.04%
+0.40%
-0.11%
+0.76%
+0.04%
+0.40%
15,034 (0)
11,714 (0)
151,615 (+2,309)
34,121 (0)
404,208 (+2,309)
11,201 (+92)
22,837 (+92)
30,231 (+92)
69,549 (0)
112,721 (+2,309)
34,057 (0)
22,561 (+92)
22,586 (+92)
27,917 (+92)
27,976 (+92)
75,173 (+1,021)
34,079 (0)
34,079 (0)
9,052 (0)
75,238 (+1,021)
34,057 (0)
34,100 (0)
0.00%
0.00%
+1.55%
0.00%
+0.57%
+0.83%
+0.40%
+0.31%
0.00%
+2.09%
0.00%
+0.41%
+0.41%
+0.33%
+0.33%
+1.38%
0.00%
0.00%
0.00%
+1.38%
0.00%
0.00%
38,026 (0)
37,131 (0)
308,158 (0)
185,273 (+2,309)
585,842 (0)
19,266 (+92)
28,110 (+92)
73,284 (0)
73,190 (0)
265,170 (0)
185,479 (+2,309)
25,886 (+92)
22,586 (+92)
70,970 (0)
27,976 (+92)
248,411 (0)
110,773 (+92)
125,813 (+1,021)
35,369 (0)
248,548 (0)
110,771 (+92)
125,854 (+1,021)
0.00%
0.00%
0.00%
+1.26%
0.00%
+0.48%
+0.33%
0.00%
0.00%
0.00%
+1.26%
+0.36%
+0.41%
0.00%
+0.33%
0.00%
+0.08%
+0.82%
0.00%
0.00%
+0.08%
+0.82%
3,114 (0)
1,546 (0)
30,090 (0)
10 (0)
1,537 (0)
265 (0)
265 (0)
264 (0)
543 (0)
1,539 (0)
10 (0)
256 (0)
256 (0)
258 (0)
256 (0)
2,337 (0)
10 (0)
10 (0)
262 (0)
2,069 (0)
10 (0)
10 (0)

@joshstevens19 joshstevens19 merged commit c58b21d into main Sep 1, 2025
18 checks passed
@donosonaumczuk donosonaumczuk deleted the fix/factory-safe-erc20 branch September 1, 2025 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants