Skip to content

Conversation

@Zodomo
Copy link
Contributor

@Zodomo Zodomo commented Jan 18, 2024

I've implemented an initial ClustersSpoke design based on ClustersHub without any of the state checks and funds management. We will likely want to implement shuttling of funds from mainnet to the spoke funds-involved function calls originate from, but that is not implemented at this time. No tests are implemented at this time.

@github-actions
Copy link

Changes to gas cost

Generated at commit: bc99b0164908700881ccf541bb84629b60c0177e, compared to commit: a30ee6777354ea26d55a34df3ce678472981ac25

🧾 Summary (100% most significant diffs)

Contract Method Avg (+/-) %
ClustersHub acceptBid
add
bidName
fundName
multicall
pokeName
reduceBid
remove
setDefaultClusterName
setWalletName
transferName
verify
+497 ❌
+656 ❌
-9,271 ✅
+378 ❌
+1,458 ❌
-11,947 ✅
+3,961 ❌
+613 ❌
+429 ❌
+399 ❌
+640 ❌
+678 ❌
+0.45%
+1.84%
-6.80%
+3.57%
+0.58%
-39.43%
+15.03%
+4.74%
+6.75%
+0.89%
+1.66%
+1.35%
Endpoint getMulticallHash
multicall
sendPayload
+94 ❌
+1,639 ❌
-142 ✅
+5.41%
+0.41%
-3.37%

Full diff report 👇
Contract Deployment Cost (+/-) Method Min (+/-) % Avg (+/-) % Median (+/-) % Max (+/-) % # Calls (+/-)
ClustersHub 4,242,513 (+303,821) acceptBid
add
bidName
fundName
multicall
pokeName
reduceBid
remove
setDefaultClusterName
setWalletName
transferName
verify
110,941 (+497)
36,269 (+656)
127,028 (-9,271)
5,013 (+247)
124,908 (+1,891)
11,050 (-19,247)
30,308 (+3,961)
13,555 (+613)
4,856 (+233)
28,326 (+269)
13,760 (+641)
51,041 (+678)
+0.45%
+1.84%
-6.80%
+5.18%
+1.54%
-63.53%
+15.03%
+4.74%
+5.04%
+0.96%
+4.89%
+1.35%
110,941 (+497)
36,269 (+656)
127,028 (-9,271)
10,963 (+378)
251,975 (+1,458)
18,350 (-11,947)
30,308 (+3,961)
13,555 (+613)
6,789 (+429)
45,412 (+399)
39,149 (+640)
51,041 (+678)
+0.45%
+1.84%
-6.80%
+3.57%
+0.58%
-39.43%
+15.03%
+4.74%
+6.75%
+0.89%
+1.66%
+1.35%
110,941 (+497)
36,269 (+656)
127,028 (-9,271)
5,013 (+247)
251,975 (+1,458)
13,050 (-17,247)
30,308 (+3,961)
13,555 (+613)
6,789 (+429)
49,985 (+269)
39,149 (+640)
51,041 (+678)
+0.45%
+1.84%
-6.80%
+5.18%
+0.58%
-56.93%
+15.03%
+4.74%
+6.75%
+0.54%
+1.66%
+1.35%
110,941 (+497)
36,269 (+656)
127,028 (-9,271)
22,863 (+639)
379,042 (+1,024)
30,950 (+653)
30,308 (+3,961)
13,555 (+613)
8,722 (+625)
57,927 (+659)
64,539 (+640)
51,041 (+678)
+0.45%
+1.84%
-6.80%
+2.88%
+0.27%
+2.16%
+15.03%
+4.74%
+7.72%
+1.15%
+1.00%
+1.35%
1 (0)
1 (0)
1 (0)
3 (0)
2 (0)
3 (+2)
1 (0)
1 (0)
2 (0)
3 (0)
2 (0)
1 (0)
Endpoint 3,732,453 (+138,981) getMulticallHash
multicall
sendPayload
1,831 (+94)
405,625 (+1,639)
2,099 (+252)
+5.41%
+0.41%
+13.64%
1,831 (+94)
405,625 (+1,639)
4,069 (-142)
+5.41%
+0.41%
-3.37%
1,831 (+94)
405,625 (+1,639)
2,099 (+252)
+5.41%
+0.41%
+13.64%
1,831 (+94)
405,625 (+1,639)
8,799 (+252)
+5.41%
+0.41%
+2.95%
1 (0)
1 (0)
17 (0)

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.

3 participants