Closed jalextowle closed 5 months ago
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contracts/src/deployers/HyperdriveDeployerCoordinator.sol | 7 | 8 | 87.5% | ||
contracts/src/deployers/ezeth/EzETHTarget5Deployer.sol | 2 | 3 | 66.67% | ||
<!-- | Total: | 38 | 40 | 95.0% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
contracts/src/internal/HyperdriveLP.sol | 1 | 96.83% | ||
contracts/src/internal/HyperdriveShort.sol | 1 | 93.81% | ||
contracts/src/internal/HyperdriveLong.sol | 2 | 93.81% | ||
contracts/src/libraries/YieldSpaceMath.sol | 2 | 91.43% | ||
contracts/src/libraries/LPMath.sol | 14 | 84.29% | ||
<!-- | Total: | 20 | --> |
Totals | |
---|---|
Change from base Build 8883450782: | -0.6% |
Covered Lines: | 1820 |
Relevant Lines: | 1957 |
Benchmark suite | Current: 37c94adfb9b1154a98a6010a3d8e079ae57664b1 | Previous: f76f5dacd38664cae6a2539770f30ff595d92141 | Deviation | Status |
---|---|---|---|---|
addLiquidity: min |
33893 gas |
33893 gas |
0% |
š° |
addLiquidity: avg |
147854 gas |
145128 gas |
1.8783% |
šØ |
addLiquidity: max |
416093 gas |
428319 gas |
-2.8544% |
ā |
checkpoint: min |
40220 gas |
40220 gas |
0% |
š° |
checkpoint: avg |
104439 gas |
104425 gas |
0.0134% |
šØ |
checkpoint: max |
211582 gas |
212154 gas |
-0.2696% |
ā |
closeLong: min |
31517 gas |
31517 gas |
0% |
š° |
closeLong: avg |
137028 gas |
138726 gas |
-1.2240% |
ā |
closeLong: max |
2640435 gas |
2640435 gas |
0% |
š° |
closeShort: min |
31394 gas |
31394 gas |
0% |
š° |
closeShort: avg |
133517 gas |
133058 gas |
0.3450% |
šØ |
closeShort: max |
253640 gas |
227565 gas |
11.4583% |
šØ |
initialize: min |
31265 gas |
31305 gas |
-0.1278% |
ā |
initialize: avg |
276084 gas |
253643 gas |
8.8475% |
šØ |
initialize: max |
344216 gas |
322898 gas |
6.6021% |
šØ |
openLong: min |
33348 gas |
33437 gas |
-0.2662% |
ā |
openLong: avg |
166838 gas |
167129 gas |
-0.1741% |
ā |
openLong: max |
253064 gas |
253057 gas |
0.0028% |
šØ |
openShort: min |
33959 gas |
33959 gas |
0% |
š° |
openShort: avg |
162757 gas |
170383 gas |
-4.4758% |
ā |
openShort: max |
407200 gas |
385545 gas |
5.6167% |
šØ |
redeemWithdrawalShares: min |
31251 gas |
31227 gas |
0.0769% |
šØ |
redeemWithdrawalShares: avg |
73611 gas |
62864 gas |
17.0956% |
šØ |
redeemWithdrawalShares: max |
204973 gas |
167572 gas |
22.3194% |
šØ |
removeLiquidity: min |
31191 gas |
31191 gas |
0% |
š° |
removeLiquidity: avg |
198467 gas |
221885 gas |
-10.5541% |
ā |
removeLiquidity: max |
395673 gas |
398887 gas |
-0.8057% |
ā |
This comment was automatically generated by workflow using github-action-benchmark.
Resolved Issues
Addresses a component of #1000.
Description
https://hackmd.io/yMNxotdqRKSuNrac9w8uFw?view
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.@jrhea
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?@mcclurejt
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?@cashd
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?