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
Use Rust version of MerkleSet #17908
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I worry a bit that leaving the python implementation of MerkleSet
in the tree might lead to someone adding a dependency to it in the future. We always have the git history if we need to go back and look at it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice to have a test where we ask for an existing addition as well, to verify the proof-of-inclusion
Purpose:
Migrates chia-blockchain to use the rust version of MerkleSet which is included in the latest version of chia_rs.
Note: it does not delete the Python version of MerkleSet just yet as we may want to keep it around a little while for posterity and delete it fully later.
Testing Notes:
The file is tested for Python compatibility in the chia_rs repository and also runs all of the tests in chia-blockchain + a new test of exclusion in
test_wallet_sync.py