Skip to content

Conversation

almogdepaz
Copy link
Contributor

@almogdepaz almogdepaz commented Aug 11, 2025

Purpose:

CHIA-3672 CHIA-3593 CHIA-3674 CHIA-3586 CHIA-3672]
build the new solver service for the new v2 plots, the solver itself is stubbed this pr is meant to include everything needed to integrate the actual solver as a new service in chia-blockchain

added solver service and all that it needs (service, cli, fixtures for tests)

changed new_signage_point_harvester to handle v2 plots -> for v2 plots the V2prover returns the partial proofs that gets sent to the farmer (instead of the full proofs in the V1 case), after that the partial proofs go to the solver service asynchronously returns the full proof

added ProverProtocol to as the interface to the provers although there are some differences in the prover apis that allows more generic code

added the needed message types:
V2QualityChains for the harvester response for v2 plots partial proofs
SolverInfo is what we send to the solver for the solve api
SolverResponse for the response from the solver

Current Behavior:

new_signage_point_harvester will asynchronously send full proofs for winning plots

New Behavior:

new_signage_point_harvester
in the case of winning v1 plots asynchronously send full proofs
in the case of winning v2 plots asynchronously send partial proofs which the farmer will use to get the full proofs with the solver

Testing Notes:

@almogdepaz almogdepaz added Blockchain team Issues tagged for Blockchain team to work on harvester Changed Required label for PR that categorizes merge commit message as "Changed" for changelog labels Aug 11, 2025
@almogdepaz almogdepaz force-pushed the new_solver_service branch 3 times, most recently from 09c5e07 to 216b4a1 Compare August 11, 2025 23:10
arvidn
arvidn previously approved these changes Aug 21, 2025
Copy link
Contributor

Copy link
Contributor

@wjblanke wjblanke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aok

@arvidn arvidn added ready_to_merge Submitter and reviewers think this is ready and removed coverage-diff labels Aug 22, 2025
@pmaslana pmaslana merged commit 52cd108 into main Aug 22, 2025
718 of 723 checks passed
@pmaslana pmaslana deleted the new_solver_service branch August 22, 2025 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blockchain team Issues tagged for Blockchain team to work on Changed Required label for PR that categorizes merge commit message as "Changed" for changelog harvester ready_to_merge Submitter and reviewers think this is ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants