-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Query Slashing Parameters #2399
Comments
I think this should use the paramstore once thats merged, and should be generalizable. More along the lines of either: |
For sure! We can still have the endpoint and update later. |
This proposal is accepted! Is there anyone working on implementation here? |
The paramstore isn't merged yet. |
And |
I think we should serve the params from the routes of the module that instantiates them so that would be |
Just want to note that we can still do that in a standardized way from params. Personally I like params coming first, but you probably have a better understanding of how clear it would be to the average user so I'm fine with |
is this still |
@ValarDragon @jackzampolin do you guys know if the param store got merged so we can move forward with the implementation of this issue? |
Param store is merged afaik. |
That would unblock this... |
Summary
Expose slashing params via Gaia-lite and CLI
Problem Definition
On voyager we were trying to track validator uptime (in this case defined as the amount of blocks they signed), which is possible with using the
signing_info
, but the problem is that we rely on thedefaultSignedBlocksWindow
to calculate the uptime on the rolling window. That means if the parameter is changed through governance clients won't be able to get the new parameter (besides of hardcoding the value).Proposal
Create a
GET /slashing/parameters
endpoint and CLI cmd to expose the paramsFor Admin Use
The text was updated successfully, but these errors were encountered: