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 zkevm_estimateGasPrice #3248

Merged
merged 4 commits into from
Feb 9, 2024
Merged

add zkevm_estimateGasPrice #3248

merged 4 commits into from
Feb 9, 2024

Conversation

tclemos
Copy link
Contributor

@tclemos tclemos commented Feb 9, 2024

Closes #3235.

What does this PR do?

add zkevm_estimateGasPrice

@tclemos tclemos added this to the v0.6.0 milestone Feb 9, 2024
@tclemos tclemos requested a review from ARR552 February 9, 2024 17:20
@tclemos tclemos self-assigned this Feb 9, 2024
@cla-bot cla-bot bot added the cla-signed label Feb 9, 2024
jsonrpc/endpoints_zkevm.go Outdated Show resolved Hide resolved
@tclemos tclemos merged commit 2e79d28 into develop Feb 9, 2024
18 checks passed
@tclemos tclemos deleted the feature/estimate-gas-price branch February 9, 2024 19:17
@agnusmor agnusmor modified the milestones: v0.6.0, v0.7.0 Feb 19, 2024
tclemos added a commit that referenced this pull request Feb 20, 2024
tclemos added a commit that referenced this pull request Feb 20, 2024
rachit77 added a commit to 0xPolygon/cdk-validium-node that referenced this pull request Feb 22, 2024
* check unsupported precompiles  (0xPolygonHermez#3264)

* check unsupported precompiles

* downgrade prover

* restore solc version

* update SC

* new errors

* fix as workaround to close batch on tx oog (0xPolygonHermez#3271)

Co-authored-by: agnusmor <agnusmor@gmail.com>

* handle executor close batch

* added sanity check closing an empty batch

* change log

---------

Co-authored-by: agnusmor <agnusmor@gmail.com>

* fix RPC closebatch executor error (0xPolygonHermez#3272)

* fix RPC closebatch executor error

* fix close batch

* fix close batch

* fix close batch

* fix close batch

* fix close batch

---------

Co-authored-by: agnusmor <agnusmor@gmail.com>

* fix executor error closebatch (0xPolygonHermez#3276)

* fix executor error closebatch

* fix linter

---------

Co-authored-by: agnusmor <agnusmor@gmail.com>

* add oog2 checks for all the oog cheks (0xPolygonHermez#3277)

* fix oog2 in internal gas estimation (0xPolygonHermez#3280)

* fix oog2

* fix oog2 for estimateGas

---------

Co-authored-by: tclemos <thiago@polygon.technology>

* execution mode (0xPolygonHermez#3285)

* execution mode

* execution mode

* execution mode

* update prover image (0xPolygonHermez#3286)

* fix log when error on batch sanity check (0xPolygonHermez#3287)

* update prover image to v4.0.13 (0xPolygonHermez#3289)

* update prover image (0xPolygonHermez#3290)

* fix deltaTimestamp when debug tx (0xPolygonHermez#3291)

* fix deltaTimestamp when debug tx

* fix deltaTimeStamp for unsigned txs execution for specific block

* fix deltaTimeStamp for unsigned txs execution for specific block

* Revert "fix deltaTimeStamp for unsigned txs execution for specific block"

This reverts commit eb77e04.

* Revert "fix deltaTimeStamp for unsigned txs execution for specific block"

This reverts commit d2cfa78.

* fix debug trace l1 info tree index and add l1 info tree data

* move MockL1InfoRoot from sequencer to state to allow multiple components to use the same value

---------

Co-authored-by: agnusmor <agnusmor@gmail.com>

* fix checkStateInconsistency when starting sequencer (0xPolygonHermez#3294)

* disable delete addrQueue if empty as a workaround (0xPolygonHermez#3295)

* add aggregator.BatchProofL1BlockConfirmations config parameter (0xPolygonHermez#3302)

* add SkipVerifyL1InfoRoot as true when debug trace needs to provide the l1 info tree data (0xPolygonHermez#3321)

* fix tx index provided in the tx log responses (0xPolygonHermez#3303)

* Fix null fields for pending blocks (0xPolygonHermez#3274)

* Add endpoint to estimate ZK Counters (0xPolygonHermez#3260)

* add zkevm_estimateGasPrice (0xPolygonHermez#3248) (0xPolygonHermez#3327)

---------

Co-authored-by: Toni Ramírez <58293609+ToniRamirezM@users.noreply.github.com>
Co-authored-by: agnusmor <agnusmor@gmail.com>
Co-authored-by: Joan Esteban <129153821+joanestebanr@users.noreply.github.com>
Co-authored-by: Thiago Coimbra Lemos <tclemos@users.noreply.github.com>
Co-authored-by: tclemos <thiago@polygon.technology>
Co-authored-by: agnusmor <100322135+agnusmor@users.noreply.github.com>
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.

create new endpoint to provide estimated effective gas price
2 participants