-
Notifications
You must be signed in to change notification settings - Fork 2
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
Include stop orders in gas calculations #1806
Conversation
witgaw
commented
Jul 3, 2023
- Include stop orders in gas calculations.
- Remove LP shapes.
8712c51
to
ac3afd6
Compare
ac3afd6
to
6cb73f6
Compare
6cb73f6
to
c536192
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to add stop orders to the calculation of gas for a batch. They are a separate "section" of the batch submission.
1acfa67
to
7573887
Compare
@@ -87,7 +87,7 @@ gas = min((maxGas/minBlockCapacity)-1,batchGas) | |||
|
|||
```go | |||
gasOliq = network.transaction.defaultgas + peg cost factor x pegs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The LP submission, amendment and cancelation should not produce any orders anymore so why do we still need this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, the gas is the same as for the the regular order submission so doesn't really matter, but I can remove that section.
4be18a0
to
5e525d9
Compare
5e525d9
to
e40724e
Compare