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

BIP158: add test vectors and generation code #668

Merged
merged 1 commit into from
Apr 8, 2018

Conversation

aakselrod
Copy link
Contributor

In this PR, we add test vectors for filter and header construction and the code to generate them. The included test vectors are for testnet with a value of 20 for P. The code generates filters and headers for values of 1 through 32 for P using testnet blocks. Currently, to run the code, the Roasbeef fork of btcd (at https://github.com/roasbeef/btcd) is required to be running locally in testnet mode; this will be changed in a future PR after the code is merged into the btcsuite mainline.

In this commit, we add test vectors for filter and header construction and
the code to generate them. The included test vectors are for testnet with a
value of 20 for P. The code generates filters and headers for values of 1
through 32 for P using testnet blocks. Currently, to run the code,
the `Roasbeef` fork of `btcd` (at https://github.com/roasbeef/btcd) is
required to be running locally in testnet mode; this will be changed in a
future commit after the code is merged into the `btcsuite` mainline.
@MattLongCT
Copy link

You certainly knocked that off the "TODO"! Code generation is properly formatted in my limited view.

@luke-jr luke-jr merged commit f7cfefa into bitcoin:master Apr 8, 2018
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.

3 participants