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

Add effective_gas_price to receipt #2222

Merged
merged 5 commits into from
Jun 26, 2023

Conversation

ToniRamirezM
Copy link
Contributor

@ToniRamirezM ToniRamirezM commented Jun 23, 2023

What does this PR do?

Adds effective_gas_price to receipt

@ToniRamirezM ToniRamirezM self-assigned this Jun 23, 2023
@cla-bot cla-bot bot added the cla-signed label Jun 23, 2023
@ToniRamirezM ToniRamirezM added this to the v0.2.0 milestone Jun 23, 2023
@ToniRamirezM ToniRamirezM changed the title Add effective_gas_used to receipt Add effective_gas_price to receipt Jun 23, 2023
@ToniRamirezM ToniRamirezM requested a review from ARR552 June 26, 2023 07:55
@ToniRamirezM ToniRamirezM merged commit 6eca7e0 into feature/new-gas-price-estimation Jun 26, 2023
15 checks passed
@ToniRamirezM ToniRamirezM deleted the fix/receiptGas branch June 26, 2023 08:29
ToniRamirezM added a commit that referenced this pull request Jun 30, 2023
* wip

Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>

* update executor proto (#2122)

* update executor proto

* gas price as string

Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>

* new smc

* smc docker image

* merging feature/HashDB

Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>

* wip

Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>

* fixing comments.

Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>

* improve: adding missing safe check for L2 Min Gas Price.

Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>

* fix: fixing eth-CallEcrecoverInvalidSignature.json 'expectedOldStateRoot'.

Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>

* improve: adding missing check in finalizer for old txs.

Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>

* Optimize trace (#2183) (#2206)

* Optimize trace (#2183)

* optimize trace

* fix memory reading

* update docker image

* update prover image

* fix converter

* fix memory

* fix step memory

* fix structlogs

* fix structlogs

* fix structlogs

* fix structlogs

* fix structlogs

* fix structlogs

* fix structlogs

* fix structlogs

* update prover image

* fix struclogs

* fix memory size

* fix memory size

* fix memory size

* refactor memory resize

* refactor memory resize

* update test

* new prover

* New image with geth1.12.0

* geth1.11.5

* fix: fixing contract addresses in configs.

Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>

* changing zkevm-mock-l1-network image to hermeznetwork/geth-zkevm-contracts:v2.0.0-RC1-fork.5-geth1.12.0

Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>

* Fix/group8 (#2211)

* fix gaslimit

* fix gaslimit

* fix gaslimit

* smc

* genBlockNumb

* use forkID while encoding/decoding txs (#2219)

* Add effective_gas_price to receipt (#2222)

* add effective_gas_used to receipt

* fix

* fix

* fix sql

* fix

* fix sql

* fix sql

* fix

* fix

* fix helper

* fix sql

* change sql

* update prover image

* update prover image

* effective gas price comparation (#2234)

* effective gas price comparation

* rename function

* fix

* refactor

* linter

* fix

* update prover image

* update prover image

* refactor

* refactor gas prices

* refactor gas prices

* fix

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* fix test

* mutex

* fix test

* do not return error on l1gas=0

* remove commented code

* refactor

* fix conflicts

* fix conflicts

* make factors floats

* fix gasprice

* fix gasprice

* merge develop and fix conflicts

---------

Signed-off-by: Nikolay Nedkov <nikolai_nedkov@yahoo.com>
Co-authored-by: Toni Ramírez <58293609+ToniRamirezM@users.noreply.github.com>
Co-authored-by: Alonso <ARR551@protonmail.com>
Co-authored-by: Toni Ramírez <toni@polygon.technology>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants