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

Document the consensus rules for Sec. 3.8 Note Commitment Trees #3319

Merged
merged 7 commits into from
Jan 14, 2022

Conversation

upbqdn
Copy link
Member

@upbqdn upbqdn commented Jan 4, 2022

Solution

This PR documents the following three consensus rules from Section 3.8 Note Commitment Trees:
image

Closes #3213

Review

Anyone can review.

@zfnd-bot zfnd-bot bot assigned upbqdn Jan 4, 2022
@codecov
Copy link

codecov bot commented Jan 4, 2022

Codecov Report

Merging #3319 (876cab0) into main (8887ed8) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3319      +/-   ##
==========================================
+ Coverage   77.95%   77.99%   +0.03%     
==========================================
  Files         266      266              
  Lines       31483    31483              
==========================================
+ Hits        24542    24554      +12     
+ Misses       6941     6929      -12     

@teor2345 teor2345 requested a review from dconnolly January 4, 2022 20:47
@teor2345
Copy link
Contributor

teor2345 commented Jan 4, 2022

The macOS failure should be fixed once we merge PRs #3168, #3292, and #3294.

@teor2345 teor2345 requested review from conradoplg and removed request for dconnolly January 4, 2022 23:54
@teor2345
Copy link
Contributor

teor2345 commented Jan 4, 2022

This looks good to me, but @conradoplg and Deirdre know this code much better than I do.

@teor2345 teor2345 added A-docs Area: Documentation P-Low labels Jan 6, 2022
Copy link
Collaborator

@conradoplg conradoplg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@upbqdn upbqdn enabled auto-merge (squash) January 13, 2022 23:06
@upbqdn upbqdn merged commit ece2d0f into main Jan 14, 2022
@upbqdn upbqdn deleted the document-commitment-trees branch January 14, 2022 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-docs Area: Documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document consensus rules from Zcash spec: Section 3.8 Note Commitment Trees
4 participants