-
Notifications
You must be signed in to change notification settings - Fork 312
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
Investigate and finalize resource pricing #658
Comments
My understanding is that for PayForData we want to measure account for the bytes but ideally we also want to add a constant cost per PfD s.t. batching is encouraged in the future. I do think this is possible and not difficult in the SDK. For the other Txs types (sdk messages) I guess we rather want clarity on how exactly they are priced and if we want to change any of that default behaviour? One message type that comes to mind are the IBC related txs (so this issue/comment is kinda related here: #235 (comment)). |
The goal of this issue to is to document the formula used for each transaction type, so that we have a concrete understanding of how gas is deducted. |
in #1029 , we're creating a script to run standard cosmos-sdk transactions on an inprocess network that has the ability to change genesis params and monitor gas. This should help us investigate gas params. |
## Overview We need to be able to modify the genesis state when using the testnode. This PR refactors the testnode to allow for that. It also includes a helper function that handles creation of this genesis state and simplify the process of starting an in process node. closes #1037 part of/blocking #658 spun out of and blocking #1029 ## Checklist - [x] New and updated code has appropriate documentation - [x] New and updated code has new and/or updated testing - [x] Required CI checks are passing - [x] Visual proof for any user facing features like CLI or documentation updates - [x] Linked issues closed with keywords Co-authored-by: Rootul P <[email protected]>
listing this as incentivized testnet stretch milestone, as we can change parameters and add docs at any time after the feature freeze without breaking anything. We also have a good idea of how much gas roughly a PFB takes up relative to other tx types, and we have code to trace the gas consumption of any transaction in any scenario should we need more precise data for a given situation. |
referencing a comment where we discuss the gas costs of IBC. Notably, following the defaults of IBC, we are not charging gas per signature verified when updating the client. We would also like to investigate the reasoning behind the decision not to count gas for signature verification in IBC. |
can close imo since we have the above analysis and have set the default min gas price to .1utia |
We need to investigate and decide on resource pricing. While we should easily be able to change the default gas configs found in tendermint and the sdk, its unclear how specific we can get as to adding multipliers to specific types of sdk txs.
The text was updated successfully, but these errors were encountered: