Skip to content
This repository was archived by the owner on Feb 17, 2025. It is now read-only.

Add effective_gas_price to receipt#2222

Merged
ToniRamirezM merged 5 commits into
feature/new-gas-price-estimationfrom
fix/receiptGas
Jun 26, 2023
Merged

Add effective_gas_price to receipt#2222
ToniRamirezM merged 5 commits into
feature/new-gas-price-estimationfrom
fix/receiptGas

Conversation

@ToniRamirezM
Copy link
Copy Markdown
Contributor

@ToniRamirezM ToniRamirezM commented Jun 23, 2023

What does this PR do?

Adds effective_gas_price to receipt

@ToniRamirezM ToniRamirezM requested a review from agnusmor June 23, 2023 14:55
@ToniRamirezM ToniRamirezM self-assigned this Jun 23, 2023
@ToniRamirezM ToniRamirezM requested a review from tclemos as a code owner June 23, 2023 14:55
@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
@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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants