Skip to content
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

feat: Support the gas estimator interface as part of CIP 18 #3287

Closed
cmwaters opened this issue Apr 2, 2024 · 0 comments
Closed

feat: Support the gas estimator interface as part of CIP 18 #3287

cmwaters opened this issue Apr 2, 2024 · 0 comments
Assignees
Labels
area:core_and_app Relationship with Core node and Celestia-App area:state Related to fetching state and state execution external Issues created by non node team members

Comments

@cmwaters
Copy link
Contributor

cmwaters commented Apr 2, 2024

Implementation ideas

This is a tracking issue for the celestia-node component of the gas estimator interface as outlined in CIP-18. Specifically, this constitutes implementing the client side, which will use the service for gas estimation and gas price estimation as part of the Blob Service.

Some noteworthy points:

  • Light (and full) nodes will use the consensus node as the default server for requesting an estimation of the gas and the gas price. A user can optionally specify a different source for obtaining these values.
  • Transactions that are purely PFBs can use the offline "modelled" way of measuring the gas and thus don't need to request from the server.
  • [Optional] We may want to come up with some lazy way of requesting the price. i.e. in non-congested times, the price becomes quite predictable and thus we may not need to request a new price every time. Instead we can react to when a transaction fails CheckTx and update our local view of the minimum gas price.
@github-actions github-actions bot added needs:triage external Issues created by non node team members labels Apr 2, 2024
@renaynay renaynay added area:state Related to fetching state and state execution area:core_and_app Relationship with Core node and Celestia-App and removed needs:triage labels Apr 4, 2024
@vgonkivs vgonkivs closed this as completed Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:core_and_app Relationship with Core node and Celestia-App area:state Related to fetching state and state execution external Issues created by non node team members
Projects
None yet
Development

No branches or pull requests

4 participants