Closed jalextowle closed 3 months ago
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contracts/src/deployers/rseth-linea/RsETHLineaHyperdriveCoreDeployer.sol | 1 | 2 | 50.0% | ||
contracts/src/deployers/rseth-linea/RsETHLineaTarget0Deployer.sol | 1 | 2 | 50.0% | ||
contracts/src/deployers/rseth-linea/RsETHLineaTarget1Deployer.sol | 1 | 2 | 50.0% | ||
contracts/src/deployers/rseth-linea/RsETHLineaTarget2Deployer.sol | 1 | 2 | 50.0% | ||
contracts/src/deployers/rseth-linea/RsETHLineaTarget3Deployer.sol | 1 | 2 | 50.0% | ||
contracts/src/deployers/rseth-linea/RsETHLineaTarget4Deployer.sol | 1 | 2 | 50.0% | ||
contracts/src/instances/rseth-linea/RsETHLineaBase.sol | 13 | 17 | 76.47% | ||
contracts/src/deployers/rseth-linea/RsETHLineaHyperdriveDeployerCoordinator.sol | 16 | 27 | 59.26% | ||
<!-- | Total: | 43 | 64 | 67.19% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
contracts/src/libraries/LPMath.sol | 1 | 84.39% | ||
<!-- | Total: | 1 | --> |
Totals | |
---|---|
Change from base Build 10621797190: | -0.6% |
Covered Lines: | 2354 |
Relevant Lines: | 2634 |
Benchmark suite | Current: dee18e64e6fc8ab25350257891b627e7d98eab47 | Previous: bad6abb89b338b39fb30749fdd9008c6728bb096 | Deviation | Status |
---|---|---|---|---|
addLiquidity: min |
33812 gas |
33812 gas |
0% |
🟰 |
addLiquidity: avg |
192939 gas |
192576 gas |
0.1885% |
🚨 |
addLiquidity: max |
474677 gas |
474677 gas |
0% |
🟰 |
checkpoint: min |
40338 gas |
40338 gas |
0% |
🟰 |
checkpoint: avg |
147737 gas |
147792 gas |
-0.0372% |
✅ |
checkpoint: max |
259386 gas |
259386 gas |
0% |
🟰 |
closeLong: min |
31390 gas |
31390 gas |
0% |
🟰 |
closeLong: avg |
129674 gas |
129843 gas |
-0.1302% |
✅ |
closeLong: max |
2539405 gas |
2539405 gas |
0% |
🟰 |
closeShort: min |
31378 gas |
31378 gas |
0% |
🟰 |
closeShort: avg |
124902 gas |
124712 gas |
0.1524% |
🚨 |
closeShort: max |
277300 gas |
277300 gas |
0% |
🟰 |
initialize: min |
31356 gas |
31356 gas |
0% |
🟰 |
initialize: avg |
355842 gas |
355912 gas |
-0.0197% |
✅ |
initialize: max |
424766 gas |
424766 gas |
0% |
🟰 |
openLong: min |
33370 gas |
33370 gas |
0% |
🟰 |
openLong: avg |
173704 gas |
173599 gas |
0.0605% |
🚨 |
openLong: max |
339743 gas |
339743 gas |
0% |
🟰 |
openShort: min |
33936 gas |
33936 gas |
0% |
🟰 |
openShort: avg |
172241 gas |
172120 gas |
0.0703% |
🚨 |
openShort: max |
414520 gas |
394598 gas |
5.0487% |
🚨 |
redeemWithdrawalShares: min |
31240 gas |
31240 gas |
0% |
🟰 |
redeemWithdrawalShares: avg |
74402 gas |
76276 gas |
-2.4569% |
✅ |
redeemWithdrawalShares: max |
311190 gas |
311190 gas |
0% |
🟰 |
removeLiquidity: min |
31246 gas |
31246 gas |
0% |
🟰 |
removeLiquidity: avg |
216819 gas |
216153 gas |
0.3081% |
🚨 |
removeLiquidity: max |
403683 gas |
403671 gas |
0.0030% |
🚨 |
This comment was automatically generated by workflow using github-action-benchmark.
Description
This PR adds support for Kelp DAO's rsETH yield source on Linea. This was a pretty standard LRT yield source, but there are a few things for reviewers to think about:
wrsETH
, but it is redeemable 1:1 with rsETH. rsETH seems like it has more name recognition, but it's worth thinking about. We could also call thisWrsETHHyperdrive
.asBase = true
when the fees are zero? We could just rip this logic out if we wanted to.