Skip to content

Commit

Permalink
Merge pull request #4 from OffchainLabs/gas-estimate-no-padding
Browse files Browse the repository at this point in the history
Document that gasEstimateL1Component doesn't pad its estimate
  • Loading branch information
PlasmaPower authored May 12, 2023
2 parents 87d5809 + 32e97f0 commit f88a36c
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/node-interface/NodeInterface.sol
Original file line number Diff line number Diff line change
Expand Up @@ -98,8 +98,10 @@ interface NodeInterface {
/**
* @notice Estimates a transaction's l1 costs.
* @dev Use eth_call to call.
* This method is exactly like gasEstimateComponents, but doesn't include the l2 component
* This method is similar to gasEstimateComponents, but doesn't include the l2 component
* so that the l1 component can be known even when the tx may fail.
* This method also doesn't pad the estimate as gas estimation normally does.
* If using this value to submit a transaction, we'd recommend first padding it by 10%.
* @param data the tx's calldata. Everything else like "From" and "Gas" are copied over
* @param to the tx's "To" (ignored when contractCreation is true)
* @param contractCreation whether "To" is omitted
Expand Down

0 comments on commit f88a36c

Please sign in to comment.