Skip to content

fix(CI): update foundry-gas-diff to version v3.21 (#1789) - #1790

Merged
MauroToscano merged 1 commit into
stagingfrom
testnet
Feb 10, 2025
Merged

fix(CI): update foundry-gas-diff to version v3.21 (#1789)#1790
MauroToscano merged 1 commit into
stagingfrom
testnet

Conversation

@MauroToscano

Copy link
Copy Markdown
Contributor

No description provided.

@MauroToscano
MauroToscano merged commit f009da5 into stagingFeb 10, 2025
@github-actions

Copy link
Copy Markdown

Changes to gas cost

Generated at commit: a3ec96d9b56a6b6e657b56f32094b76e08f6c039, compared to commit: 306f72d2c0f03f0107f55725c2ba5fa168f39fc4

🧾 Summary (10% most significant diffs)

ContractMethodAvg (+/-)%
RegistryCoordinatorHarnessblsApkRegistry+332 ❌+116.90%
Slasherinitialize+470 ❌+103.98%
StakeRegistryHarnessdelegation+332 ❌+101.84%
AlignedLayerServiceManagerbatchesState+688 ❌+95.29%

Full diff report 👇
ContractDeployment Cost (+/-)MethodMin (+/-)%Avg (+/-)%Median (+/-)%Max (+/-)%# Calls (+/-)
RegistryCoordinatorHarness10,130,048 (+4,267,886)blsApkRegistry
initialize
stakeRegistry
616 (+332)
56,281,475 (+1,577,892)
682 (+332)
+116.90%
+2.88%
+94.86%
616 (+332)
56,281,475 (+1,577,892)
682 (+332)
+116.90%
+2.88%
+94.86%
616 (+332)
56,281,475 (+1,577,892)
682 (+332)
+116.90%
+2.88%
+94.86%
616 (+332)
56,281,475 (+1,577,892)
682 (+332)
+116.90%
+2.88%
+94.86%
6 (0)
6 (0)
6 (0)
Slasher1,598,813 (+762,834)initialize922 (+470)+103.98%922 (+470)+103.98%922 (+470)+103.98%922 (+470)+103.98%6 (0)
StakeRegistryHarness5,139,510 (+2,005,505)delegation
initializeQuorum
658 (+332)
146,200 (+3,117)
+101.84%
+2.18%
658 (+332)
165,996 (+3,117)
+101.84%
+1.91%
658 (+332)
166,100 (+3,117)
+101.84%
+1.91%
658 (+332)
166,100 (+3,117)
+101.84%
+1.91%
6 (0)
1,152 (0)
AlignedLayerServiceManager8,615,423 (+3,384,189)batchesState
createNewTask
disableVerifier
disabledVerifiers
enableVerifier
isVerifierDisabled
receive
setDisabledVerifiers
1,410 (+688)
57,794 (+1,711)
24,299 (+390)
514 (+131)
24,191 (+390)
856 (+275)
23,496 (+179)
24,250 (+451)
+95.29%
+3.05%
+1.63%
+34.20%
+1.64%
+47.33%
+0.77%
+1.90%
1,410 (+688)
77,704 (+1,643)
35,816 (+315)
514 (+131)
24,760 (+315)
1,522 (+275)
47,097 (+182)
35,176 (+370)
+95.29%
+2.16%
+0.89%
+34.20%
+1.29%
+22.05%
+0.39%
+1.06%
1,410 (+688)
77,958 (+1,717)
35,816 (+315)
514 (+131)
24,760 (+315)
856 (+275)
47,472 (+277)
35,176 (+370)
+95.29%
+2.25%
+0.89%
+34.20%
+1.29%
+47.33%
+0.59%
+1.06%
1,410 (+688)
78,804 (+1,810)
47,334 (+240)
514 (+131)
25,329 (+240)
2,856 (+275)
47,472 (+277)
46,103 (+290)
+95.29%
+2.35%
+0.51%
+34.20%
+0.96%
+10.65%
+0.59%
+0.63%
256 (0)
256 (0)
2 (0)
1 (0)
2 (0)
3 (0)
256 (0)
2 (0)
ProxyAdmin757,646 (+329,348)upgrade
upgradeAndCall
39,641 (+845)
56,632,018 (+1,579,612)
+2.18%
+2.87%
39,650 (+845)
56,632,018 (+1,579,612)
+2.18%
+2.87%
39,653 (+845)
56,632,018 (+1,579,612)
+2.18%
+2.87%
39,653 (+845)
56,632,018 (+1,579,612)
+2.18%
+2.87%
24 (0)
6 (0)
AVSDirectory3,174,782 (+1,463,555)initialize100,637 (+2,514)+2.56%100,637 (+2,514)+2.56%100,637 (+2,514)+2.56%100,637 (+2,514)+2.56%6 (0)
ServiceManagerMock2,632,771 (+1,054,437)initialize73,362 (+1,409)+1.96%73,362 (+1,409)+1.96%73,362 (+1,409)+1.96%73,362 (+1,409)+1.96%6 (0)
BLSApkRegistryHarness3,124,237 (+1,317,862)initializeQuorum
setBLSPublicKey
45,978 (+772)
90,107 (+740)
+1.71%
+0.83%
45,978 (+772)
90,107 (+740)
+1.71%
+0.83%
45,978 (+772)
90,107 (+740)
+1.71%
+0.83%
45,978 (+772)
90,107 (+740)
+1.71%
+0.83%
1,152 (0)
6 (0)
TransparentUpgradeableProxy897,999 (+341,241)fallback1,544 (+477)+44.70%48,066 (+774)+1.64%8,110 (+477)+6.25%120,000 (+885)+0.74%30 (0)
IndexRegistry1,707,545 (+637,885)initializeQuorum45,592 (+528)+1.17%45,592 (+528)+1.17%45,592 (+528)+1.17%45,592 (+528)+1.17%1,152 (0)
StrategyManagerMock2,401,372 (+1,154,891)setAddresses89,742 (+597)+0.67%89,742 (+597)+0.67%89,742 (+597)+0.67%89,742 (+597)+0.67%6 (0)

Sign up for freeto 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.

2 participants

@MauroToscano@JuArce