-
Notifications
You must be signed in to change notification settings - Fork 618
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
2.3.0 #12182
2.3.0 #12182
Conversation
### Protocol Changes | ||
**No Changes** | ||
|
||
### Non-protocol Changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to mention the state sync changes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, somehow just forgot it. Added it now
CHANGELOG.md
Outdated
### Protocol Changes | ||
* Sets `chunk_validator_only_kickout_threshold` to 70. Uses this kickout threshold as a cutoff threshold for contribution of endorsement ratio in rewards calculation: if endorsement ratio is above 70%, the contribution of endorsement ratio in average uptime calculation is 100%, otherwise it is 0%. Endorsements received are now included in `BlockHeader` to improve kickout and reward calculation for chunk validators. | ||
* Changes the structure of ChunkEndorsements |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should come under non-protocol changes section. We are changing the network message tho, but that isn't a part of protocol I believe?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah i just added it without looking too deeply because I saw it's included in the protocol features but you're right, def not strictly a protocol change. Actually let's just delete this part. I think it's prob not super relevant for most node operators.
It's not strictly a protocol change and most operators aren't going to find it relevant
Bumps version strings and changelog messages for 2.3.0-rc.1