Skip to content

switch adjustFeesForGas to use sdk.Coins #2411

@ValarDragon

Description

@ValarDragon

Summary

For the mempool fees, the adjustFeesForGas mechanism is currently dividing the amount of gas by the number of atoms per unit gas. Its more intuitive to change this to gasPrice, e.g. 1 gas = gasPrice coins. Its easiest to represent the gasPrice as sdk.Coins. We need to run through expected gas range of txs (perhaps 100 to 10 million), and expected reasonable price in atoms (perhaps .0001 atoms to 10 atoms?), and run through the logic just to ensure its all handled cleanly.

ref #2376, /cc @jaekwon

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions