-
Notifications
You must be signed in to change notification settings - Fork 40
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(consensus): HeaviestSubtreeForkChoice #505
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of discards from funcs that return void. Also do you think we could add agave permalinks on analogous methods and key logic?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good the me :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice
This PR implements part of the HeaviestSubtreeForkChoice. Specifically part that has been identified as being used in the ReplayStage and does not depend on components not implemented yet (ie Bank, BankFork etc)
The main method identified as being used in the ReplayStage and implemented in this PR are
Apart from the above the following public methods were implemented, either because they were needed as part of the above listed methods or needed for testing.
The PR also contains various private methods which were implemented either because they are needed in the above public methods or needed in tests