Validator cut [WIP] #2052
Validator cut [WIP] #2052
Conversation
open-junius
added
the
skip-cargo-audit
|
Where are these coming from? Two days ago, we were enabling a childkey recycling of 18%, then it was cancelled. Now placing vali dividends in subnet hands? Is there any public discourse on this? |
| #[pallet::type_value] | ||
| /// Default validator cut 50% | ||
| pub fn DefaultValidatorCut<T: Config>() -> u16 { | ||
| u16::MAX / 2 |
There was a problem hiding this comment.
this should default to 100% else you're just forcing a massive change on all the subnet owners
There was a problem hiding this comment.
The cut is rate of split between validators and miners. You can check code in line 669. If it is 100%, miners get nothing.
There was a problem hiding this comment.
You’re right that makes sense, I misunderstood the default setting.
It helps shitnets prop up their token prices. |
|
@open-junius is there a limit to how high the cut can be? There should be. |
|
Description
It will allow the subnet owner to set the validator cut during emission.
Related Issue(s)
Type of Change
Breaking Change
If this PR introduces a breaking change, please provide a detailed description of the impact and the migration path for existing applications.
Checklist
cargo fmtandcargo clippyto ensure my code is formatted and linted correctlyScreenshots (if applicable)
Please include any relevant screenshots or GIFs that demonstrate the changes made.
Additional Notes
Please provide any additional information or context that may be helpful for reviewers.