Skip to content

Conversation

@jalextowle
Copy link
Contributor

This update should make adding yield source integrations easier and more seamless.

@github-actions
Copy link

Hyperdrive Gas Benchmark

Benchmark suite Current: e44b71d Previous: 46c834d Deviation Status
addLiquidity: min 33893 gas 1546 gas 2092.3027% 🚨
addLiquidity: avg 146578 gas 67936 gas 115.7589% 🚨
addLiquidity: max 402392 gas 293159 gas 37.2607% 🚨
checkpoint: min 29254 gas 1182 gas 2374.9577% 🚨
checkpoint: avg 107851 gas 48477 gas 122.4787% 🚨
checkpoint: max 210817 gas 191559 gas 10.0533% 🚨
closeLong: min 31450 gas 1492 gas 2007.9088% 🚨
closeLong: avg 101741 gas 29374 gas 246.3641% 🚨
closeLong: max 228760 gas 152303 gas 50.2006% 🚨
closeShort: min 31327 gas 1494 gas 1996.8541% 🚨
closeShort: avg 97591 gas 33444 gas 191.8042% 🚨
closeShort: max 184441 gas 149156 gas 23.6564% 🚨
initialize: min 31283 gas 1451 gas 2055.9614% 🚨
initialize: avg 250651 gas 213962 gas 17.1474% 🚨
initialize: max 322113 gas 253953 gas 26.8396% 🚨
openLong: min 33503 gas 1499 gas 2135.0233% 🚨
openLong: avg 130351 gas 51658 gas 152.3346% 🚨
openLong: max 252131 gas 185562 gas 35.8743% 🚨
openShort: min 33870 gas 1520 gas 2128.2895% 🚨
openShort: avg 138875 gas 51482 gas 169.7545% 🚨
openShort: max 362083 gas 181325 gas 99.6873% 🚨
redeemWithdrawalShares: min 31205 gas 1488 gas 1997.1102% 🚨
redeemWithdrawalShares: avg 72358 gas 22375 gas 223.3877% 🚨
redeemWithdrawalShares: max 151504 gas 109301 gas 38.6117% 🚨
removeLiquidity: min 31213 gas 1530 gas 1940.0654% 🚨
removeLiquidity: avg 222473 gas 151225 gas 47.1139% 🚨
removeLiquidity: max 366958 gas 325781 gas 12.6395% 🚨

This comment was automatically generated by workflow using github-action-benchmark.

Copy link
Contributor

@jrhea jrhea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. it'd be good to add an integrations readme that outlines the process of wrapping new yield sources

@jalextowle jalextowle merged commit 45dd879 into main Mar 11, 2024
@jalextowle jalextowle deleted the jalextowle/feature/simplify-integrations branch March 11, 2024 16:48
dpaiton pushed a commit that referenced this pull request Mar 28, 2024
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