Conversation
| "adagioTime": 0 | ||
| }, | ||
| "baseFeePerGas": "0x2c", | ||
| "baseFeePerGas": "0x28048c5ec000", |
There was a problem hiding this comment.
We should adjust this to ensure transfers are < ~$0.0001.
@danrobinson to confirm this value.
There was a problem hiding this comment.
<$0.001 (tenth of a cent, not hundredth)! And should be slightly less
There was a problem hiding this comment.
should we go with 0.04 wei per gas? would mean that min tx cost is 21000 * 0.04 = $0.00084
There was a problem hiding this comment.
What's the gas cost of a transaction that includes a TIP-20 token transfer, though?
There was a problem hiding this comment.
There was a problem hiding this comment.
likely to change soon once we start charging for storage reads properly, with policy/paused checks might be closer to 50k
543e3c8 to
fcdf35d
Compare
mattsse
left a comment
There was a problem hiding this comment.
lgtm, only have doc nits
| use tempo_primitives::TempoHeader; | ||
|
|
||
| pub const TEMPO_BASE_FEE: u64 = 44; | ||
| pub const TEMPO_BASE_FEE: u64 = 44_000_000_000_000; |
There was a problem hiding this comment.
can we add the unit here and document and also add the hex value in docs (helps me find stuff)
this reduces mental load when reviewing this
Closes tempoxyz#629 changes all relevant code to calculate transaction cost as `(gas * gas_price).div_ceil(1e12)`
Closes #629
changes all relevant code to calculate transaction cost as
(gas * gas_price).div_ceil(1e12)