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

feat(prover): tie capacity to a specific block id #413

Merged
merged 10 commits into from
Sep 28, 2023
Merged

Conversation

cyberhorsey
Copy link
Contributor

@cyberhorsey cyberhorsey commented Sep 28, 2023

capacity can be released twice if, when submitting a proof, the block becomes verified by someone else (ie: you were just too late to submit).

Because there are too many edge cases with arbitrary capacity, it should be tied to a block ID, and only be able to be released once for a given block.

cmd/flags/prover.go Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Sep 28, 2023

Codecov Report

Merging #413 (560b7e6) into main (f192e0a) will decrease coverage by 0.29%.
The diff coverage is 4.76%.

@@            Coverage Diff             @@
##             main     #413      +/-   ##
==========================================
- Coverage   50.72%   50.44%   -0.29%     
==========================================
  Files          39       39              
  Lines        4962     5000      +38     
==========================================
+ Hits         2517     2522       +5     
- Misses       2189     2223      +34     
+ Partials      256      255       -1     
Files Coverage Δ
prover/config.go 88.59% <100.00%> (+0.10%) ⬆️
prover/server/api.go 30.00% <0.00%> (ø)
prover/prover.go 31.70% <22.22%> (-0.13%) ⬇️
prover/capacity_manager/capacity_manager.go 0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes

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

@cyberhorsey cyberhorsey enabled auto-merge (squash) September 28, 2023 02:59
@cyberhorsey cyberhorsey merged commit bdca930 into main Sep 28, 2023
4 checks passed
@cyberhorsey cyberhorsey deleted the capacity_block_id branch September 28, 2023 02:59
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