Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Cost analysis of a council election full block #2782

Merged
merged 6 commits into from Apr 10, 2021

Conversation

kianenigma
Copy link
Contributor

Current value is 166_000 DOLLARS. With the current value of DOT, this is more than a million. But I expect the DOT to DOLLARS ratio to be tuned soon, therefore let's assume DOLLARS is actually 1USD.

In this case, I think it is a bit too low. Current VotingBaseBond is 20 * DOLLARS + ... only for the storage item. I would say we increase this to 50 or 100 DOLLARS to reach a safety net of half a million DOLLARS at least.

@kianenigma kianenigma added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit. labels Mar 31, 2021
@shawntabrizi
Copy link
Contributor

Just as a note, I think we want to deprecate the use of "DOLLAR" in our code base since it implies things which you mention clearly do not apply above.

It might make sense to come up with another unit of measurement. Maybe # of Big Macs?

https://www.economist.com/big-mac-index

@bkchr
Copy link
Member

bkchr commented Mar 31, 2021

An attack on Polkadot costs 500 Big macs :D

@shawntabrizi
Copy link
Contributor

@kianenigma some unused warning

@kianenigma
Copy link
Contributor Author

Just as a note, I think we want to deprecate the use of "DOLLAR" in our code base since it implies things which you mention clearly do not apply above.

It might make sense to come up with another unit of measurement. Maybe # of Big Macs?

https://www.economist.com/big-mac-index

I haven't had time to look into this yet, but the current measure is also not too bad. We've never changed these constants in the past, so I am a bit worried about the consequences there. But, assuming that they will have no consequence, and we can easily update them, I think this model is easy and simple enough.

For example, the governance bodies can re-evaluate this once per quarter based on the mean value of DOT, and just change this one number.

I'm curious to see what happens with #2451.

@kianenigma
Copy link
Contributor Author

bot merge

@ghost
Copy link

ghost commented Apr 9, 2021

Error: Missing process info; check that the PR belongs to a project column.

Merge cannot succeed as it is. Check out the criteria for merge.

@bkchr
Copy link
Member

bkchr commented Apr 9, 2021

@kianenigma you need to merge master

@kianenigma
Copy link
Contributor Author

bot merge

@ghost
Copy link

ghost commented Apr 9, 2021

Error: Missing process info; check that the PR belongs to a project column.

Merge cannot succeed as it is. Check out the criteria for merge.

@kianenigma
Copy link
Contributor Author

@kianenigma you need to merge master

@bkchr you need to approve

@bkchr
Copy link
Member

bkchr commented Apr 9, 2021

@kianenigma you need to merge master

@bkchr you need to approve

Done 😜

@kianenigma kianenigma merged commit 028d4b6 into master Apr 10, 2021
@kianenigma kianenigma deleted the kiz-council-election-cost branch April 10, 2021 04:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants