Skip to content

EIP1559: RLP parser for DynamicFeeTx #166

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.

Sign up for GitHub

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

Jump to bottom
Closed
raventid wants to merge 15 commits into from
Closed

EIP1559: RLP parser for DynamicFeeTx #166

raventid wants to merge 15 commits into from

Conversation

@raventid
Copy link

@raventid raventid commented Aug 18, 2023

No description provided.

@raventid raventid self-assigned this Aug 18, 2023
@raventid raventid force-pushed the EIP2930 branch 2 times, most recently from 0d1935f to 031a865 Compare August 25, 2023 08:34
@raventid raventid force-pushed the EIP1559-rlp-parser branch 3 times, most recently from 7691853 to a054b3f Compare August 29, 2023 08:49
@raventid raventid marked this pull request as ready for review August 29, 2023 08:54
@raventid raventid added the fullTestSuite Run OZ tests and part of dapps tests label Aug 29, 2023
@raventid raventid requested a review from anton-lisanin August 29, 2023 09:06
@raventid raventid force-pushed the EIP1559-rlp-parser branch 6 times, most recently from 7466e96 to 0733728 Compare September 1, 2023 08:39
Base automatically changed from EIP2930 to develop September 1, 2023 09:54
kristinaNikolaevaa and others added 3 commits September 4, 2023 13:31
@kristinaNikolaevaa @raventid
added tests for dynamic_gas transactions
0ddc0f3 
(cherry picked from commit 9580e29)
@raventid
EIP-1559: RLP parser implementation
91c4b93 
(cherry picked from commit c41c3c9)
@raventid
add skipped dynamicfee match
1252c17 
(cherry picked from commit 6717a92)
@raventid raventid removed the fullTestSuite Run OZ tests and part of dapps tests label Sep 4, 2023
@kristinaNikolaevaa kristinaNikolaevaa added the fullTestSuite Run OZ tests and part of dapps tests label Sep 4, 2023
@kristinaNikolaevaa kristinaNikolaevaa added extendedFullTestSuite and removed fullTestSuite Run OZ tests and part of dapps tests labels Sep 25, 2023
@neonlabsorg neonlabsorg deleted a comment from neonlabstech Oct 4, 2023
@neonlabstech
Copy link

neonlabstech commented Oct 4, 2023

Dapps report

Cost report for "Curve" dApp

Action Fee Cost in $ Accounts TRx Estimated Gas Used Gas Used % of EG
Metapool: add liquidity 3.6956943454005757 0.28004693039202055 22 229 14207556 10106080 71.13
Metapool: swap 1 0.8421387022826 0.06382490905700514 19 230 5873516 2300000 39.16
Metapool: swap 2 0.85208666681164 0.06455751126605043 19 233 5906516 2330000 39.45
Metapool: withdraw admin fees 0.0036417916073 0.0002759526074007252 15 1 49500 10000 20.2
Pool: add liquidity 2.657937772167678 0.2014022046369455 26 577 11359524 7287280 64.15
Pool: swap 1 0.99147648109152 0.07509827560661836 19 272 6335516 2720000 42.93
Pool: swap 2 0.99492321452944 0.07530655394476923 19 272 6335516 2720000 42.93
Pool: swap 3 0.9952087258792 0.07532816453295187 19 272 6335516 2720000 42.93
Pool: withdraw admin fees 0.00365317339238 0.000276468509161926 16 1 60500 10000 16.53
Stablecoin: transfer 0.00364540107363 0.0002757969010746801 10 1 33000 10000 30.3
Tri pool: add liquidity 3.63834858461853 0.2752914251708029 3 39 15444572 9948040 64.41
Tri pool: swap 1 0.1352119326201 0.010231843900865085 20 37 3750516 370000 9.87
Tri pool: swap 2 0.13495501568865 0.010212402318401564 20 37 3750516 370000 9.87
Tri pool: swap 3 0.13141199246544 0.00994462486986089 20 36 3739516 360000 9.63
Tri pool: apply new fee 0.00365193055245 0.0002762908961078235 8 1 33000 10000 30.3
Tri pool: withdraw admin fees 0.00365717616732 0.00027757458762471543 18 1 60500 10000 16.53
gauge v3: deposit 2.3141614312877343 0.1754064940073164 28 1 15191572 6308800 41.53
gauge controller: vote 1.752609329040519 0.13284195095220563 26 1 20264596 4791520 23.64
gauge controller: vote 1.1976290549799706 0.09066966934796383 26 1 18595588 3274240 17.61
gauge controller: vote 1.197328316108004 0.09075015871169213 26 1 18595588 3274240 17.61
gauge controller: vote 1.1133386367797669 0.08447953455598484 26 1 20264596 3044560 15.02
gauge controller: vote 1.7524193315543972 0.1327927817759112 26 1 20264596 4791520 23.64
gauge controller: vote 1.197255571636882 0.09067923905409872 26 1 18595588 3274240 17.61
gauge controller: vote 1.7520627738374561 0.13269992044336856 26 1 20264596 4791520 23.64
gauge controller: vote 1.197255571636882 0.0907273567555228 26 1 18595588 3274240 17.61
gauge controller: vote 1.1951907073167514 0.09054102662979854 26 1 18595588 3274240 17.61

Cost report for "Uniswap V3" dApp

Action Fee Cost in $ Accounts TRx Estimated Gas Used Gas Used % of EG
Token approve 0.5568936341504023 0.04213691131308287 9 1 1552280 1527280 98.39
NonfungiblePositionManager - Mint position 6.293618576756154 0.47651472223958663 4 28 38247000 17260240 45.13
NonfungiblePositionManager - Increase liquidity 0.08386513007084 0.006349760264927076 23 23 9338680 230000 2.46
NonfungiblePositionManager - Decrease Liquidity 0.2548916544553043 0.01933708518009794 17 1 13800520 699040 5.07
NonfungiblePositionManager - Collect Fees 1.1101409582977244 0.08406705597430389 20 1 3119560 3044560 97.6
NonfungiblePositionManager - Burn Liquidity Position 0.08739475815382144 0.006615895931482302 15 1 7641400 239680 3.14
Direct swap 0.00364631000308 0.0002763136892602993 17 1 185000 10000 5.41
Burn transaction 0.08739475815382144 0.006620519114188639 11 1 4606840 239680 5.2
Collect transaction 0.5568936341504023 0.04216775765147846 12 1 1562280 1527280 97.76

Cost report for "Uniswap V2" dApp

Action Fee Cost in $ Accounts TRx Estimated Gas Used Gas Used % of EG
Token approve 0.5562167154821953 0.042101122255275396 9 1 1552280 1527280 98.39
Direct swap 0.00364187781862 0.00027556956829495946 18 1 135000 10000 7.41

@kristinaNikolaevaa kristinaNikolaevaa mentioned this pull request Oct 4, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@anton-lisanin anton-lisanin Awaiting requested review from anton-lisanin

Assignees

@raventid raventid

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@raventid @neonlabstech @anton-lisanin @kristinaNikolaevaa