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

Fetch stake table #2237

Closed
wants to merge 3 commits into from
Closed

Fetch stake table #2237

wants to merge 3 commits into from

Conversation

tbro
Copy link
Contributor

@tbro tbro commented Oct 30, 2024

Closes EspressoSystems/HotShot#3726

This PR

Adds the ability to fetch stake table from l1. It adds StakeCommittee type which replaces StaticCommittee. This new type implements Membershp trait, holds L1Client and fn update_stake_table to update Self.stake_table. In this way, hotshot will be able to obtain StakeTable via SystemContextHandle.members.

TODO

  • The actual http request to l1 has yet to be implemented.
  • Names are provisional

@tbro tbro force-pushed the tb/pos/fetch-stake-table branch 4 times, most recently from 4e71be7 to c9c334d Compare October 31, 2024 21:23
@tbro tbro changed the title Tb/pos/fetch stake table Fetch stake table Oct 31, 2024
@tbro tbro force-pushed the tb/pos/fetch-stake-table branch from c9c334d to 5448181 Compare November 1, 2024 23:05
tbro added 2 commits November 4, 2024 13:49
Adds the ability to fetch stake table from l1. It adds StakeCommittee
type which replaces StaticCommittee. This new type implements
Membershp trait, holds L1Client and fn update_stake_table to update
Self.stake_table. In this way, hotshot will be able to obtain
StakeTable via SystemContextHandle.members.
@tbro tbro force-pushed the tb/pos/fetch-stake-table branch from 5448181 to 6ce37b5 Compare November 4, 2024 20:26
&self,
_block: u64,
_address: Address,
) -> Vec<<TYPES::SignatureKey as SignatureKey>::StakeTableEntry> {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

link

tbro added a commit that referenced this pull request Dec 5, 2024
Branched from #2325. Adds most everything from #2237. Main blocker now is to iron out
`ChainConfig` versions. Then we can get into the actual stake table
retrieval logic.
@tbro
Copy link
Contributor Author

tbro commented Dec 5, 2024

Superseded by #2361

@tbro tbro closed this Dec 5, 2024
tbro added a commit that referenced this pull request Dec 6, 2024
Branched from #2325. Adds most everything from #2237. Main blocker now is to iron out
`ChainConfig` versions. Then we can get into the actual stake table
retrieval logic.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[EPOCH TRANSITION] - Membership trait has an additional method to fetch a stake table for a given epoch
1 participant