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

Stateless witness builder #29719

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

jwasinger
Copy link
Contributor

@jwasinger jwasinger commented May 7, 2024

Rebase of the witness builder on top of #29519 . Apparently I cannot re-open a closed pull request whose branch has been force pushed to.

This PR introduces necessary functionality to construct block witnesses. It doesn't include construction in block building, or cross-validation.

Only the blockchain test suite is modified with the introduction a stateless test. The test executes blockchain tests, constructing and verifying a proof for each block.

@jwasinger jwasinger marked this pull request as ready for review May 8, 2024 22:40
@jwasinger jwasinger marked this pull request as draft May 9, 2024 05:01
… in a new test suite which executes blockchain tests, building and verifying stateless proofs for each test block.
@jwasinger jwasinger marked this pull request as ready for review May 14, 2024 06:50
@jwasinger
Copy link
Contributor Author

jwasinger commented May 14, 2024

This adds a new test suite TestStatelessBlockchain which adds a large amount of running time to CI. We may consider disabling it as a last step before merging this when it is approved.

@jwasinger jwasinger marked this pull request as draft May 14, 2024 18:05
@jwasinger
Copy link
Contributor Author

jwasinger commented May 14, 2024

Converted back to draft while I address remaining TODOs (parts of the code are confusing rn).

@jwasinger jwasinger marked this pull request as ready for review May 16, 2024 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant