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

stake-contract: shift eligibility when slashing newcomers provisioners #1363

Merged
merged 3 commits into from
Feb 14, 2024

Conversation

herr-seppia
Copy link
Member

@herr-seppia herr-seppia commented Feb 8, 2024

Slashing a provisioner results in its reward to be decreased.
If there is no accrued reward, it's eligibility is shifted to the next epoch

@herr-seppia herr-seppia changed the base branch from master to refactor-sortition February 8, 2024 15:46
@herr-seppia herr-seppia self-assigned this Feb 8, 2024
Base automatically changed from refactor-sortition to master February 9, 2024 14:06
@herr-seppia herr-seppia force-pushed the shift-stake-epoch branch 4 times, most recently from 2aee7f8 to b480d96 Compare February 14, 2024 11:19
@herr-seppia herr-seppia changed the title WIP: Shift stake eligibility consensus: shift eligibility when slashing newcomers provisioners Feb 14, 2024
@herr-seppia herr-seppia changed the title consensus: shift eligibility when slashing newcomers provisioners stake-contract: shift eligibility when slashing newcomers provisioners Feb 14, 2024
@herr-seppia herr-seppia marked this pull request as ready for review February 14, 2024 13:23
HDauven
HDauven previously approved these changes Feb 14, 2024
Copy link
Member

@HDauven HDauven left a comment

Choose a reason for hiding this comment

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

LGTM

- Change `slash` to not touch the staked amount
- Change `slash` to shift eligibility when reward is 0
- Add `hard_slash` to remove stake amount portion
@ureeves ureeves merged commit c35d52c into master Feb 14, 2024
6 checks passed
@ureeves ureeves deleted the shift-stake-epoch branch February 14, 2024 15:44
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.

None yet

3 participants