Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added a pause script to test-utils #644

Merged
merged 4 commits into from
Nov 1, 2023

Conversation

jalextowle
Copy link
Contributor

Adds a rust script for pausing the pool. This is the first rust script that we've used for protocol related tasks, and it's a good example for future scripts.

@jalextowle jalextowle requested a review from jrhea October 31, 2023 17:22
Copy link

github-actions bot commented Oct 31, 2023

Hyperdrive Gas Benchmark

Benchmark suite Current: acf50e1 Previous: 1148b95 Deviation Status
addLiquidity: min 733 gas 733 gas 0% 🟰
addLiquidity: avg 54102 gas 54072 gas 0.0555% 🚨
addLiquidity: max 96720 gas 96704 gas 0.0165% 🚨
checkpoint: min 558 gas 558 gas 0% 🟰
checkpoint: avg 47628 gas 47642 gas -0.0294%
checkpoint: max 99410 gas 99410 gas 0% 🟰
closeLong: min 755 gas 755 gas 0% 🟰
closeLong: avg 23630 gas 23397 gas 0.9959% 🚨
closeLong: max 114183 gas 114183 gas 0% 🟰
closeShort: min 802 gas 802 gas 0% 🟰
closeShort: avg 24832 gas 25012 gas -0.7197%
closeShort: max 111483 gas 111387 gas 0.0862% 🚨
initialize: min 662 gas 662 gas 0% 🟰
initialize: avg 177327 gas 177327 gas 0% 🟰
initialize: max 251633 gas 251633 gas 0% 🟰
openLong: min 757 gas 757 gas 0% 🟰
openLong: avg 56897 gas 56892 gas 0.0088% 🚨
openLong: max 218401 gas 218401 gas 0% 🟰
openShort: min 712 gas 712 gas 0% 🟰
openShort: avg 56203 gas 56235 gas -0.0569%
openShort: max 217803 gas 217803 gas 0% 🟰
removeLiquidity: min 777 gas 777 gas 0% 🟰
removeLiquidity: avg 75980 gas 75957 gas 0.0303% 🚨
removeLiquidity: max 203187 gas 203187 gas 0% 🟰

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

@coveralls
Copy link
Collaborator

coveralls commented Oct 31, 2023

Coverage Status

coverage: 97.551%. remained the same when pulling acf50e1 on jalextowle/competition/pause-script into 1148b95 on main.

Copy link
Contributor

@jrhea jrhea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good. It would be nice if the readme had an example of how to run these bin scripts

Copy link
Member

@ryangoree ryangoree left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

crates/test-utils/src/bin/pause.rs Show resolved Hide resolved
@jalextowle jalextowle enabled auto-merge (squash) October 31, 2023 19:13
@jalextowle jalextowle merged commit 3c9bcea into main Nov 1, 2023
6 checks passed
@jalextowle jalextowle deleted the jalextowle/competition/pause-script branch November 1, 2023 00:15
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