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

feat: add documentation on block processing #221

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bowenwang1996
Copy link
Collaborator

No description provided.

@render
Copy link

render bot commented Jun 16, 2021

- Chunk headers root computed from chunk headers does not match `chunk_headers_root` in the header.
- Transactions root computed from chunk headers does not match `chunk_tx_root` in the header.
- For some index `i`, `chunk_mask[i]` does not match whether a new chunk from shard `i` is included in the block.
- Its vrf output is invalid
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Created #222 to track this

@frol frol added the WG-protocol Protocol Standards Work Group should be accountable label Sep 5, 2022
@bowenwang1996 bowenwang1996 removed the WG-protocol Protocol Standards Work Group should be accountable label Sep 28, 2022
@frol frol added the WG-unknown No Work Group is currently accountable label Oct 7, 2022
@frol
Copy link
Collaborator

frol commented Dec 20, 2022

@bowenwang1996 Should this be merged eventually? Can you reassign the reviewer?

@frol frol added S-draft/needs-author-revision A NEP in the DRAFT stage that needs an author revision. A-Nomicon-Changes An update to the Nomicon documentation. labels Dec 20, 2022
Copy link
Collaborator Author

@bowenwang1996 bowenwang1996 left a comment

Choose a reason for hiding this comment

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

I think it is still relevant. @mm-near could you review (or find someone else to review) this PR? Thanks!

@ori-near ori-near added S-review/needs-wg-to-assign-sme A NEP that needs working group to assign two SMEs. and removed S-draft/needs-author-revision A NEP in the DRAFT stage that needs an author revision. labels Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Nomicon-Changes An update to the Nomicon documentation. S-review/needs-wg-to-assign-sme A NEP that needs working group to assign two SMEs. WG-unknown No Work Group is currently accountable
Projects
Status: DRAFT
Development

Successfully merging this pull request may close these issues.

None yet

3 participants