-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Add EIP: Separate gas type for calldata #8552
Conversation
✅ All reviewers have approved. |
The commit 513d424 (as a parent of 5ea7da6) contains errors. |
Co-authored-by: Andrew B Coathup <28278242+abcoathup@users.noreply.github.com>
Co-authored-by: Andrew B Coathup <28278242+abcoathup@users.noreply.github.com>
Co-authored-by: Etan Kissling <etan@status.im>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All Reviewers Have Approved; Performing Automatic Merge...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
VersoriumX
@@ -63,7 +63,7 @@ def get_max_fees(tx: Transaction) -> [int, int, int]: | |||
elif tx.type == BLOB_TX_TYPE: | |||
return [tx.max_fee_per_gas, tx.max_fee_per_blob_gas, tx.max_fee_per_gas] | |||
elif is_eip_1559(tx.type): | |||
return [tx.max_fee_per_gas, 0, max_fee_per_gas] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
commit 9eb41928548665e15617b27d258ac13379390acc
Add a new type of gas for transaction calldata. Add a new transaction type that provides max_basefee and priority_fee as a vector, providing values for execution gas, blob gas and calldata gas. Modify EIP-1559 to use the same mechanism for the three types of gas.