Update token parameters and fundamental units #400
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Sep 5, 2022
tzemanovic
force-pushed
the
tomas/pos-integration
branch
2 times, most recently
from
600b2ed to
ee19f2f
Compare
This adds a new feature "async-client" and "tendermint-rpc" to shared crate that if enabled generate async client code for all router's handler methods, and implements it for tendermint-rpc client, respectively.
* tomas/rpc-queries-router: wasm: update checksums changelog: add #553 apps: replace RPC module and its handlers with new queries mod shared: add new queries router macro to replicate handwritten RPC paths add deps for router macro and update `Cargo.lock`s protocol: update imports and add missing rustdoc move ledger's protocol module into shared crate
* tomas/rpc-sub-shell: [ci] wasm checksums update router: add `with_options` for handlers that use request/response move the current RPC patterns under "shell" sub-router
* tomas/rpc-sub-vp-pos: [ci] wasm checksums update queries: add more PoS queries and use them for gov RPC: add PoS is_validator and bond_amount queries shared: implement PosReadOnly for Storage
tzemanovic
force-pushed
the
brent/update-token-params
branch
from
0550abc to
76b37b7
Compare
|
both issues are closed as design decision has changed |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #391 and #386.
Based on #379.
Previously, fundamental token units were such that 10^6 fundamental units (namnam) comprised a whole NAM token. This is changed to 10^9. The proof-of-stake parameter
votes_per_tokenis also changed such that now it is 1/100 votes per fundamental unit, rather than 1/1000 votes per whole token.