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

Adjust TdS leader efficiency calculation #7766

Closed
mvines opened this issue Jan 13, 2020 · 2 comments
Closed

Adjust TdS leader efficiency calculation #7766

mvines opened this issue Jan 13, 2020 · 2 comments
Labels
stale [bot only] Added to stale content; results in auto-close after a week.
Milestone

Comments

@mvines
Copy link
Member

mvines commented Jan 13, 2020

Currently in TdS leaders are penalized if they skip any of their slots. However not all skipped slots are bad, and determining whether a skipped slot is good or bad would require much more introspection RPC facilities.

In lieu of building out more RPC APIs, a more fair efficiency calculation given the currently available information is be:
(any block produced over a leader slot segment) / (number of leader slot segments)
where "leader slot segment" is a contiguous sequence of leader slots.

Update Ramp TPS to use this when it figures the next stake delegations at the end of each round. Also consider adding this calculation solana show-block-production for visibility.

@stale
Copy link

stale bot commented Jan 30, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the stale [bot only] Added to stale content; results in auto-close after a week. label Jan 30, 2021
@stale
Copy link

stale bot commented Feb 7, 2021

This stale issue has been automatically closed. Thank you for your contributions.

@stale stale bot closed this as completed Feb 7, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
stale [bot only] Added to stale content; results in auto-close after a week.
Projects
None yet
Development

No branches or pull requests

1 participant