Closed sentilesdal closed 2 months ago
Benchmark suite | Current: 8c825339257b35203135ce448e3320867872921e | Previous: a55506bfa3b5371bf36b5678692fe30c8fbad4e2 | Deviation | Status |
---|---|---|---|---|
addLiquidity: min |
33827 gas |
33827 gas |
0% |
π° |
addLiquidity: avg |
156305 gas |
156018 gas |
0.1840% |
π¨ |
addLiquidity: max |
428417 gas |
428417 gas |
0% |
π° |
checkpoint: min |
40292 gas |
40292 gas |
0% |
π° |
checkpoint: avg |
142307 gas |
142331 gas |
-0.0169% |
β |
checkpoint: max |
253424 gas |
253424 gas |
0% |
π° |
closeLong: min |
31361 gas |
31361 gas |
0% |
π° |
closeLong: avg |
135500 gas |
135337 gas |
0.1204% |
π¨ |
closeLong: max |
2625796 gas |
2625796 gas |
0% |
π° |
closeShort: min |
31349 gas |
31349 gas |
0% |
π° |
closeShort: avg |
131743 gas |
131610 gas |
0.1011% |
π¨ |
closeShort: max |
401915 gas |
262998 gas |
52.8206% |
π¨ |
initialize: min |
31371 gas |
31371 gas |
0% |
π° |
initialize: avg |
330366 gas |
330372 gas |
-0.0018% |
β |
initialize: max |
396259 gas |
396259 gas |
0% |
π° |
openLong: min |
33370 gas |
33370 gas |
0% |
π° |
openLong: avg |
173357 gas |
173373 gas |
-0.0092% |
β |
openLong: max |
306657 gas |
306657 gas |
0% |
π° |
openShort: min |
33936 gas |
33936 gas |
0% |
π° |
openShort: avg |
167925 gas |
168009 gas |
-0.0500% |
β |
openShort: max |
415379 gas |
415307 gas |
0.0173% |
π¨ |
redeemWithdrawalShares: min |
31251 gas |
31251 gas |
0% |
π° |
redeemWithdrawalShares: avg |
74736 gas |
74854 gas |
-0.1576% |
β |
redeemWithdrawalShares: max |
209828 gas |
209828 gas |
0% |
π° |
removeLiquidity: min |
31301 gas |
31301 gas |
0% |
π° |
removeLiquidity: avg |
208698 gas |
208930 gas |
-0.1110% |
β |
removeLiquidity: max |
403640 gas |
403652 gas |
-0.0030% |
β |
This comment was automatically generated by workflow using github-action-benchmark.
Totals | |
---|---|
Change from base Build 9039156905: | 0.0% |
Covered Lines: | 1833 |
Relevant Lines: | 1966 |
Resolved Issues
Description
The testnet EzEth instance of Hyperdrive was returning vaultSharePrice values way below what it should. This now sets the price to basically: (totalSupply + interest) / totalSupply. I checked this on the currently deployed Sepolia contracts and it has sane values so I think this is all we need to do.
Review Checklists
Please check each item before approving the pull request. While going through the checklist, it is recommended to leave comments on items that are referenced in the checklist to make sure that they are reviewed. If there are multiple reviewers, copy the checklists into sections titled
## [Reviewer Name]
. If the PR doesn't touch Solidity and/or Rust, the corresponding checklist can be removed.[[Reviewer Name]]
approve
calls useforceApprove
?transfer
calls usesafeTransfer
?transferFrom
calls usemsg.sender
as thefrom
address?call
,delegatecall
,staticcall
,transfer
,send
)success
boolean checked to handle failed calls?delegatecall
, are there strict access controls on the addresses that can be called? It shouldn't be possible todelegatecall
arbitrary addresses, so the list of possible targets should either be immutable or tightly controlled by an admin.nonReentrant
?payable
functions restricted to avoid stuck ether?Safe
functions are altered, are potential underflows and overflows caught so that a failure flag can be thrown?