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

NU5 Tracking: Pre-NU5 Spec work #1872

Closed
13 tasks done
mpguerra opened this issue Mar 9, 2021 · 19 comments
Closed
13 tasks done

NU5 Tracking: Pre-NU5 Spec work #1872

mpguerra opened this issue Mar 9, 2021 · 19 comments
Labels
Epic Zenhub Label. Denotes a theme of work under which related issues will be grouped

Comments

@mpguerra
Copy link
Contributor

mpguerra commented Mar 9, 2021

This epic will track the minimum set of outstanding spec items, defined prior to NU5 and still active once NU5 is activated, which will need to be implemented in Zebra.

This tracking issue contains segments for the whole spec that includes Orchard changes for now. A checkbox means that that part of the specification describes something to implement.

A checked box or strikethrough means that we:

  • have implemented that item
  • are tracking that item in another Epic, or
  • have otherwise resolved that item.

Our mandatory Canopy checkpoint significantly reduces the number of features that we have to implement.

Leaf nodes are marked by – and correspond to implementable items. They should be followed by issue references that track the status of that issue, or a short explanation.

@mpguerra
Copy link
Contributor Author

Added #267 under "3.1. Payment Addresses and Keys", I'm guessing this might change to only implementing Sapling key derivation?

@mpguerra mpguerra added the Epic Zenhub Label. Denotes a theme of work under which related issues will be grouped label Mar 10, 2021
@mpguerra
Copy link
Contributor Author

Added #269 under "3.2. Notes"
Skipping #181 for the Sprout note encryption/decryption for now unless we really need to do this

@mpguerra
Copy link
Contributor Author

Added #250 under "3.2.1 Note Plaintexts and Memo Fields"

@mpguerra
Copy link
Contributor Author

mpguerra commented Mar 10, 2021

Added #290 #958 and #983 to "3.4. Transactions and Treestates"

@mpguerra mpguerra changed the title NU5 Tracking: Pre-NU5 work NU5 Tracking: Pre-NU5 Spec work Mar 10, 2021
@mpguerra
Copy link
Contributor Author

Added #1287 to " 3.8. Note Commitment Trees"

@mpguerra
Copy link
Contributor Author

mpguerra commented Mar 10, 2021

Added "3.10. Block Subsidy, Funding Streams, and Founders’ Reward" and related issue: #338

@mpguerra
Copy link
Contributor Author

Added #1835 and #1645 to "4.1.12. Zero-Knowledge Proving System"

@mpguerra
Copy link
Contributor Author

Added #181 to "4.7.1. Sending Notes (Sprout)"

@mpguerra
Copy link
Contributor Author

Added #269 to "4.7.2. Sending Notes (Sapling and Orchard)"

@mpguerra
Copy link
Contributor Author

Added #1377 to "4.10. SIGHASH Transaction Hashing"

@mpguerra
Copy link
Contributor Author

Added #319 to "4.15. Spend Authorization Signature (Sapling and Orchard)"

@mpguerra
Copy link
Contributor Author

mpguerra commented Mar 10, 2021

@ZcashFoundation/zebra-team I think this list is now finished, I'm just missing some tickets for:

  • 4.8.1. Dummy Notes (Sprout)
  • 4.8.2. Dummy Notes (Sapling and Orchard)
  • 4.9. Merkle Path Validity
  • 4.11 Non-malleability (Sprout)
  • 4.12. Balance (Sprout)
  • 7.10. Payment of Funding Streams

@teor2345
Copy link
Contributor

  • 4.12. Balance (Sprout)

I think this is ZIP-209, but we also need to do Sapling and Orchard balances.

Also note that a non-negative transparent pool balance is a consensus rule as well.

  • 7.10. Payment of Funding Streams

This is #801.

@mpguerra
Copy link
Contributor Author

mpguerra commented Mar 16, 2021

Removed a bunch of issues from the Epic relating to zebra-client/wallet support work (#250, #267, #269, #271, #272, #319, #320, #321, #325)

@teor2345
Copy link
Contributor

I added #801 and #196 for transaction validation. Zebra parses pre-NU5 transactions correctly, but semantic and contextual transaction validation is incomplete.

@teor2345
Copy link
Contributor

I added #477 for block header validation. Zebra parses pre-NU5 block headers correctly, but its semantic and contextual validation is incomplete.

@mpguerra
Copy link
Contributor Author

Cleaning up this tracking issue to cut scope and remove tracking redundancies:

@mpguerra
Copy link
Contributor Author

Removed #196 from the epic to cut scope and remove tracking redundancies. All issues under #196 that should be implemented are already tracked in this issue or #1854

@mpguerra mpguerra mentioned this issue Mar 23, 2021
4 tasks
@mpguerra
Copy link
Contributor Author

I think we're all done here! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Epic Zenhub Label. Denotes a theme of work under which related issues will be grouped
Projects
None yet
Development

No branches or pull requests

2 participants