Skip to content

Addressed the pre-release comments#647

Merged
jalextowle merged 1 commit intomainfrom
jalextowle/pre-release/0.0.17
Nov 1, 2023
Merged

Addressed the pre-release comments#647
jalextowle merged 1 commit intomainfrom
jalextowle/pre-release/0.0.17

Conversation

@jalextowle
Copy link
Copy Markdown
Contributor

Fixes: #633.

@jalextowle jalextowle requested a review from jrhea November 1, 2023 16:38
@jalextowle jalextowle force-pushed the jalextowle/pre-release/0.0.17 branch from 9c8c0b3 to eba8c31 Compare November 1, 2023 16:41
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 1, 2023

Hyperdrive Gas Benchmark

Benchmark suite Current: eba8c31 Previous: ee68411 Deviation Status
addLiquidity: min 733 gas 733 gas 0% 🟰
addLiquidity: avg 54102 gas 54041 gas 0.1129% 🚨
addLiquidity: max 96720 gas 96720 gas 0% 🟰
checkpoint: min 558 gas 558 gas 0% 🟰
checkpoint: avg 47632 gas 47647 gas -0.0315%
checkpoint: max 99410 gas 99410 gas 0% 🟰
closeLong: min 755 gas 755 gas 0% 🟰
closeLong: avg 23403 gas 23676 gas -1.1531%
closeLong: max 114183 gas 114183 gas 0% 🟰
closeShort: min 802 gas 802 gas 0% 🟰
closeShort: avg 26757 gas 26423 gas 1.2641% 🚨
closeShort: max 102184 gas 112872 gas -9.4691%
initialize: min 662 gas 662 gas 0% 🟰
initialize: avg 177327 gas 177288 gas 0.0220% 🚨
initialize: max 251633 gas 251633 gas 0% 🟰
openLong: min 757 gas 757 gas 0% 🟰
openLong: avg 56882 gas 56965 gas -0.1457%
openLong: max 218431 gas 218431 gas 0% 🟰
openShort: min 712 gas 712 gas 0% 🟰
openShort: avg 56297 gas 56184 gas 0.2011% 🚨
openShort: max 217803 gas 217803 gas 0% 🟰
removeLiquidity: min 777 gas 777 gas 0% 🟰
removeLiquidity: avg 75984 gas 75957 gas 0.0355% 🚨
removeLiquidity: max 203187 gas 203187 gas 0% 🟰

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

@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented Nov 1, 2023

Coverage Status

coverage: 97.551%. remained the same when pulling eba8c31 on jalextowle/pre-release/0.0.17 into ee68411 on main.

@jalextowle jalextowle enabled auto-merge (squash) November 1, 2023 17:11
@jalextowle jalextowle merged commit e3658bb into main Nov 1, 2023
@jalextowle jalextowle deleted the jalextowle/pre-release/0.0.17 branch November 1, 2023 17:48
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.

Pre-v0.0.17

3 participants