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

KIP6 logic #609

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

KIP6 logic #609

wants to merge 56 commits into from

Conversation

freshair18
Copy link

@freshair18 freshair18 commented Dec 4, 2024

Contains most consensus layer infrastracture of Kip6 - with the goal of allowing users to generate and store everlasting proofs of tx inclusion or tx acceptance.
Kip6 requires adding an extra field to the header, this will be covered in a future PR,
for now the required new field is locally simulated with a test only storage entry called pchmr_store.
New functionality is generally not called to save for testing, but a single placeholder call to calc_pchmr_root is made when building block template (with its value discarded) as a placeholder and in order to see if it has adverse effect on anything.

@freshair18 freshair18 changed the title merge with master KIP6 logic Dec 4, 2024
@freshair18 freshair18 marked this pull request as draft December 4, 2024 20:10
1) create a witness for inclusion of a leaf in a merkle tree
2) verify the inclusion given a leaf and said witness
improved tests
removed legacy code
improved general structure of code
simplified test code
added ignore for the perfomance dependant test test_writer_reentrance per instructions of Aspect and Michael
Fixed error:changed daa score to blue score for posterity blocks
minor refactoring
get_pre_posterity
get_post_posterity
get posterity_by_bscore.
Changed/improved implementation of representative_log_parents
More error logic
bug fix regarding accepted txs
refactoring a few for loops
split representative parents finding function to two parts,
removed unnecessary typing
removed commented out code
…er: the header is required but will not be available in storage during verification
…discovered; added pochm functionalities to consensus_api
bug fixes and improvements for BFS iterator
legacy_pochm complete
@freshair18 freshair18 marked this pull request as ready for review December 5, 2024 08:48
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.

1 participant