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.

Sign up for GitHub

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

Jump to bottom

Finish initial Authority implementation #134

Merged
merged 4 commits into from
Dec 6, 2018
Merged

Finish initial Authority implementation #134

merged 4 commits into from
Dec 6, 2018

Conversation

ilblackdragon
Copy link
Member

@ilblackdragon ilblackdragon commented Dec 4, 2018 •
edited
Loading

Ref #7

  • Thresholded Proof of Stake
  • CLI for staking

ilblackdragon
ilblackdragon commented Dec 5, 2018
View reviewed changes
core/beacon/src/authority.rs Outdated
);

// Shuffle proposals.
let mut seed_usize = vec![];
Copy link
Member Author

Choose a reason for hiding this comment

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

@frol is there a better way to do this?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Does seed.as_ref().iter().map(|i| i as usize) work?

@ilblackdragon ilblackdragon force-pushed the authority-2 branch 2 times, most recently from 91cb344 to ae7e742 Compare December 5, 2018 21:48
@ilblackdragon ilblackdragon changed the title [WIP] Cont. Authority Finish initial Authority implementation Dec 5, 2018
bowenwang1996
bowenwang1996 approved these changes Dec 6, 2018
View reviewed changes
core/beacon/src/authority.rs Outdated
);

// Shuffle proposals.
let mut seed_usize = vec![];
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does seed.as_ref().iter().map(|i| i as usize) work?

@ilblackdragon ilblackdragon merged commit 7f18486 into master Dec 6, 2018
@ilblackdragon ilblackdragon deleted the authority-2 branch December 6, 2018 15:10
@ilblackdragon ilblackdragon added this to the AlphaNet milestone Mar 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bowenwang1996 bowenwang1996 bowenwang1996 approved these changes

@azban azban Awaiting requested review from azban

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
AlphaNet
Development

Successfully merging this pull request may close these issues.
2 participants
@ilblackdragon @bowenwang1996