-
Notifications
You must be signed in to change notification settings - Fork 401
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
Use blob gas instead of count #6229
Conversation
724b7d9
to
708a00a
Compare
708a00a
to
a96f931
Compare
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.
I don't like it, makes it less clear. Why the change?
|
But |
The current reason is that blob count did not change with custom max blob gas settings for gnosis which led to a block production issue |
2815182
to
f149583
Compare
f149583
to
37aee8b
Compare
All blob calculations should use blob gas and do not rely on max blob count to be consistent
Changes
Types of changes
What types of changes does your code introduce?
Testing
Requires testing
If yes, did you write tests?