-
Notifications
You must be signed in to change notification settings - Fork 15
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
Total Balance in node info API #868
Comments
I am a little bit confused about this feature; it also can be served by explorer like 3rd party service. |
The total balance could indeed be calculated by third parties since we know all variables. The calculations will be pure, as they will always result in the same output based on those variables. In its current state, third parties would need to do manual updates when inflation through PF occurs. A developer must know the information below before he can manually add the inflation:
For each new inflation, a developer would have to manually add it to the other inflation calculations, unless that information becomes available through some endpoint. The block inflation, for example, is currently available through the GET https://mainnet.blockchainos.org endpoint:
Information like this allows third parties to calculate rewards in an automated way (the only thing missing in the above example is the start block height of inflation). If an endpoint or json object would be added to get all inflation variables (for block inflation, PF inflation, ...), third parties could automatically calculate the inflation at any given block height. The question is, would it be worth the hassle to actually build and provide such endpoints / json objects to calculate the total supply if the API itself can actually do that? It might be worth it if you ever need to be able to provide information about all past, current and present inflations. It basically comes down to where you want to put the responsibility of calculating the supply:
|
Total increased balance = Initial balance + block inflation + PF inflation
The text was updated successfully, but these errors were encountered: