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

feat(prover): flag for proving unassigned proofs or not #314

Merged
merged 7 commits into from
Jul 13, 2023

Conversation

cyberhorsey
Copy link
Contributor

@cyberhorsey cyberhorsey commented Jul 12, 2023

this pr adds a flag to the prover, which can determien whether this prover should prove unasigned proofs or not.

unassigned proofs:

  1. one where the proofWindow expires and the assignedprover did not submit a proof on time
  2. one where the protocol was out of capacity, and the assignedprover is address(0), which means anyone can submit proof.

This can be useful because if you are proving an unassigned proof, and you get assigned a proof, you will not start generating it until the expired proof is done if you are at proving capacity, and you may potentially be slashed if you are late delivering the proof. So you may want to configure two provers on two different machines, one set to true, one set to false, to ensure you have a prover always working on your assigned proofs.

Also improves logging in these areas which was lacking.

@cyberhorsey cyberhorsey changed the title feat(prover): flag for checking expired proofs or not feat(prover): flag for proving unassigned proofs or not Jul 12, 2023
@codecov
Copy link

codecov bot commented Jul 12, 2023

Codecov Report

Merging #314 (7d8d3c3) into main (b0b4c25) will increase coverage by 0.01%.
The diff coverage is 6.25%.

@@            Coverage Diff             @@
##             main     #314      +/-   ##
==========================================
+ Coverage   50.86%   50.88%   +0.01%     
==========================================
  Files          36       36              
  Lines        3910     3950      +40     
==========================================
+ Hits         1989     2010      +21     
- Misses       1666     1683      +17     
- Partials      255      257       +2     
Impacted Files Coverage Δ
prover/prover.go 43.01% <4.25%> (+1.22%) ⬆️
prover/config.go 72.00% <100.00%> (+0.37%) ⬆️

... and 1 file with indirect coverage changes

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

@davidtaikocha davidtaikocha merged commit 13e6d1d into main Jul 13, 2023
5 checks passed
@davidtaikocha davidtaikocha deleted the expired_proof_flag branch July 13, 2023 12:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants