Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

saturation metric fix #384

Open
gabririgo opened this issue Apr 17, 2021 · 0 comments
Open

saturation metric fix #384

gabririgo opened this issue Apr 17, 2021 · 0 comments

Comments

@gabririgo
Copy link

the saturation metric uses

getGlobalStakeByStatus("delegated")

which is different from the metric used for staking rewards calculations (totalWeightedStake), which is

aggregatedStatsByEpoch(current epoch)[3]

I believe the choice was made in order to display saturation as a projective measure for the upcoming epoch, taking into account new stake which has occurred before the start of the next epoch.

There's tradeoffs in either solution, and I think it's worth a discussion to understand whether the issue is relevant. I am happy to close the issue should we deem the projective measure, though incorrect, is better from a users' perspective.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant