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

fix!: burned commitment mmr calculation and cucumber prune mode #4453

Merged
merged 3 commits into from
Aug 11, 2022

Conversation

SWvheerden
Copy link
Collaborator

@SWvheerden SWvheerden commented Aug 11, 2022

mmr header calculation for burned outputs
Fix cucumber prune mode

Description

Fixes the MMR root calculation for burned commitments. They were calculated correctly in the database and prune mode sync, but not during header sync calculation.
Fixes pruned mode cucumber tests. Consensus values between local net and Esmeralda were not matching.
Reduces local net header version back to 0 to match Esmeralda version.

How Has This Been Tested?

Unit and cucumber tests

fixes: #4360
fixes: #4443

Fix cucumber prune mode
Fix cucumber header versions
stringhandler
stringhandler previously approved these changes Aug 11, 2022
@stringhandler stringhandler merged commit 0c062f3 into tari-project:development Aug 11, 2022
@SWvheerden SWvheerden deleted the sw_emse_fix branch August 12, 2022 05:33
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.

Investigate header version usage in cucumber Prune mode does not work
2 participants