Skip to content
This repository has been archived by the owner on May 11, 2024. It is now read-only.

feat(prover): normal prover should wait targetProofTime before submitting proofs #232

Merged
merged 9 commits into from
May 19, 2023

Conversation

RogerLamTd
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented May 18, 2023

Codecov Report

Merging #232 (7e3eba6) into main (252959f) will increase coverage by 0.04%.
The diff coverage is 65.71%.

@@            Coverage Diff             @@
##             main     #232      +/-   ##
==========================================
+ Coverage   54.39%   54.43%   +0.04%     
==========================================
  Files          36       36              
  Lines        3528     3549      +21     
==========================================
+ Hits         1919     1932      +13     
- Misses       1355     1361       +6     
- Partials      254      256       +2     
Impacted Files Coverage Δ
prover/proof_producer/dummy_producer.go 89.74% <0.00%> (-4.86%) ⬇️
prover/proof_producer/proof_producer.go 0.00% <ø> (ø)
prover/proof_producer/special_proof_producer.go 8.16% <0.00%> (-0.18%) ⬇️
prover/prover.go 53.15% <16.66%> (-0.61%) ⬇️
prover/proof_producer/zkevm_rpcd_producer.go 31.34% <88.00%> (+3.70%) ⬆️

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@RogerLamTd RogerLamTd marked this pull request as ready for review May 19, 2023 03:56
@davidtaikocha davidtaikocha requested a review from cyberhorsey May 19, 2023 04:24
@davidtaikocha davidtaikocha merged commit 2128ddc into main May 19, 2023
@davidtaikocha davidtaikocha deleted the prover-wait-proof-time branch May 19, 2023 04:38
vhjiang pushed a commit to taikoverse/taiko-degen-client that referenced this pull request Jul 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

A normal prover should at least wait protocolConfig.targetProofTime seconds before submitting proofs
2 participants