Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Comments

Companion PR for substrate gav-lazy-rewards#791

Closed
gui1117 wants to merge 1 commit intomasterfrom
gui-companion-pr-gav-lazy-rewards
Closed

Companion PR for substrate gav-lazy-rewards#791
gui1117 wants to merge 1 commit intomasterfrom
gui-companion-pr-gav-lazy-rewards

Conversation

@gui1117
Copy link
Contributor

@gui1117 gui1117 commented Jan 23, 2020

this fix the incoming change of staking API introduced in paritytech/substrate#4474

@gavofyork gavofyork added the B0-silent Changes should not be mentioned in any release notes label Feb 4, 2020
@gui1117 gui1117 force-pushed the gui-companion-pr-gav-lazy-rewards branch from 1497ab3 to 75d4f87 Compare March 3, 2020 14:15
// 28 eras in which slashes can be cancelled (7 days).
pub const SlashDeferDuration: staking::EraIndex = 28;
pub const RewardCurve: &'static PiecewiseLinear<'static> = &REWARD_CURVE;
pub const MaxNominatorRewardedPerValidator: u32 = 64;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this number is a proposition as the one used in node-runtime in substrate, though I can't really tell about the precise value to use here.

@gui1117
Copy link
Contributor Author

gui1117 commented Mar 5, 2020

superseeded by #878

@gui1117 gui1117 closed this Mar 5, 2020
@gui1117 gui1117 deleted the gui-companion-pr-gav-lazy-rewards branch March 5, 2020 12:01
tomusdrw pushed a commit that referenced this pull request Mar 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

B0-silent Changes should not be mentioned in any release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants