Skip to content

Conversation

@aleph-v
Copy link
Contributor

@aleph-v aleph-v commented May 22, 2023

Fees can be at most 100% to prevent revert errors.

@coveralls
Copy link
Collaborator

coveralls commented May 22, 2023

Coverage Status

Coverage: 81.491%. Remained the same when pulling fbce2da on violet/feature/feeLimits into 3cd095e on main.

@jalextowle jalextowle force-pushed the violet/feature/feeLimits branch from c473a3a to 3c2f84c Compare May 22, 2023 20:47
@github-actions
Copy link

github-actions bot commented May 22, 2023

Hyperdrive Gas Benchmark

Benchmark suite Current: fbce2da Previous: 05e3d58 Deviation Status
addLiquidity: min 711 gas 711 gas 0% 🟰
addLiquidity: avg 43095 gas 44736 gas -3.6682%
addLiquidity: max 81028 gas 81028 gas 0% 🟰
checkpoint: min 492 gas 492 gas 0% 🟰
checkpoint: avg 23044 gas 23044 gas 0% 🟰
checkpoint: max 33513 gas 33513 gas 0% 🟰
closeLong: min 637 gas 637 gas 0% 🟰
closeLong: avg 44691 gas 43793 gas 2.0506% 🚨
closeLong: max 78327 gas 79240 gas -1.1522%
closeShort: min 616 gas 616 gas 0% 🟰
closeShort: avg 35778 gas 35841 gas -0.1758%
closeShort: max 78750 gas 78750 gas 0% 🟰
initialize: min 667 gas 667 gas 0% 🟰
initialize: avg 138216 gas 138216 gas 0% 🟰
initialize: max 139696 gas 139696 gas 0% 🟰
openLong: min 685 gas 685 gas 0% 🟰
openLong: avg 109130 gas 108263 gas 0.8008% 🚨
openLong: max 147584 gas 147584 gas 0% 🟰
openShort: min 727 gas 727 gas 0% 🟰
openShort: avg 136808 gas 137910 gas -0.7991%
openShort: max 206459 gas 206459 gas 0% 🟰
removeLiquidity: min 547 gas 547 gas 0% 🟰
removeLiquidity: avg 57337 gas 59713 gas -3.9790%
removeLiquidity: max 111079 gas 111079 gas 0% 🟰

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

@jalextowle jalextowle force-pushed the violet/feature/feeLimits branch from 3c2f84c to 1687b2d Compare May 22, 2023 21:02
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.

4 participants