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

Update randomness for conformance tipsets and update V7 proof verification #911

Merged
merged 3 commits into from
Jan 4, 2021

Conversation

austinabell
Copy link
Contributor

@austinabell austinabell commented Jan 2, 2021

Summary of changes
Changes introduced in this pull request:

  • Converts from sector size to proof type based on network version (part of Implement Calico (v7) upgrade logic #910)
  • Uses recorded randomness for tipset vectors (is used now with extracted tipset vectors)

This is branching off #907 because it makes a cleaner merge, otherwise would have to break previous version or bump minor version of types and assume this change comes in. Also, somewhat based on #903 because the thing marked as deprecated is removed in that PR (and would only be used in V1 actors, which is why deprecating for now instead of removing)

Reference issue to close (if applicable)

Closes

Other information and links

@austinabell austinabell marked this pull request as draft January 2, 2021 21:43
Base automatically changed from austin/cratereplace to main January 4, 2021 18:41
@austinabell austinabell marked this pull request as ready for review January 4, 2021 21:08
@austinabell austinabell added Spec Change Updates to implementation required due to a Filecoin spec change after implementation Status: Needs Review and removed Status: DO NOT MERGE labels Jan 4, 2021
@austinabell austinabell merged commit dd173e4 into main Jan 4, 2021
@austinabell austinabell deleted the austin/cfproof branch January 4, 2021 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Spec Change Updates to implementation required due to a Filecoin spec change after implementation Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants