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

Whisk: assert zeroed values during selection gap #3425

Merged
merged 2 commits into from
Jun 15, 2023

Conversation

dapplion
Copy link
Collaborator

Whisk requires to not submit tracker shuffling during a selection gap. To have more deterministic block production the spec should assert whisk_shuffle_proof_M_commitment and whisk_shuffle_proof to be zero. It should also assert whisk_post_shuffle_trackers to be zero instead of pre_shuffle_trackers. Setting the values to pre_shuffle_trackers does not add any new information while increasing the number of non-zero bytes in the block increasing its compressed size.

@asn-d6
Copy link
Contributor

asn-d6 commented Jun 14, 2023

I think the idea we had with @JustinDrake here is that the proposer would create a valid shuffle proof but would use the identity permutation (i.e. not permute at all). We did this so that the consensus logic does not have a too complicated special-case during that selection gap.

If you think that the proposer should instead send nothing during selection gap, we should strongly consider it.

@dapplion
Copy link
Collaborator Author

I my PoC I have to already consider the selection gap in the block production logic. This change does not add any more complexity IMO

@asn-d6
Copy link
Contributor

asn-d6 commented Jun 14, 2023

I my PoC I have to already consider the selection gap in the block production logic. This change does not add any more complexity IMO

Aight, makes sense. can you also update the obsolete comment regarding unchanged trackers, please?

Copy link
Contributor

@asn-d6 asn-d6 left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

@asn-d6 asn-d6 merged commit 834f6f7 into ethereum:dev Jun 15, 2023
24 checks passed
dapplion added a commit to dapplion/consensus-specs that referenced this pull request Jun 16, 2023
* Assert zeroed values during selection gap

* Update comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants