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

Gas profile for liquidations #1758

Closed
wants to merge 6 commits into from
Closed

Gas profile for liquidations #1758

wants to merge 6 commits into from

Conversation

leomassazza
Copy link
Contributor

Adds a test (skipped by default) to execute a gas profiling on liquidations with multiple markets.
It's configured to create 100 markets and run liquidations for an account with 1, 2, 5, 10, 50 and 100 positions.

@leomassazza leomassazza self-assigned this Aug 10, 2023
@codecov
Copy link

codecov bot commented Aug 10, 2023

Codecov Report

Merging #1758 (9ae13b8) into main (fa1857c) will not change coverage.
The diff coverage is n/a.

❗ Current head 9ae13b8 differs from pull request most recent head ac4175f. Consider uploading reports for the commit ac4175f to get more accurate results

@@           Coverage Diff           @@
##             main    #1758   +/-   ##
=======================================
  Coverage   72.77%   72.77%           
=======================================
  Files          57       57           
  Lines         720      720           
  Branches      236      236           
=======================================
  Hits          524      524           
  Misses        167      167           
  Partials       29       29           
Flag Coverage Δ
core-contracts 93.26% <ø> (ø)
core-modules 90.47% <ø> (ø)
core-utils 68.57% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@leomassazza
Copy link
Contributor Author

leomassazza commented Aug 10, 2023

Results of first run

Raw data

| Positions | Open Position | Liquidate| 
| 1         | 606785        | 1084296  |
| 2         | 694748        | 1252236  |
| 5         | 999646        | 1751220  |
| 10        | 1508239       | 2623424  |
| 50        | 5596230       | 9815584  |
| 100       | 10754348      | 19014610 |

Results

Open Position

Base 508,652
Per Position 98,133

Liquidation

Base 911,298
Per Position 172,998

@leomassazza leomassazza closed this Oct 4, 2023
@noisekit noisekit deleted the pv3-gasprofile-l1 branch November 15, 2023 22:23
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.

1 participant