Skip to content
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

Set eip4844 blob limits to 1/2 #24

Merged
merged 2 commits into from
Oct 18, 2023
Merged

Set eip4844 blob limits to 1/2 #24

merged 2 commits into from
Oct 18, 2023

Conversation

dapplion
Copy link
Member

@dapplion dapplion commented Oct 11, 2023

Justification

Impact of the block size on Gnosis network performance, and reasoning for 1/2 choice: https://hackmd.io/@5qNKk0aeQlygax4hX3rVXw/SkY3dBV-6

Big blocks experiment summary

photo_2023-10-05_08-21-52

Noticeable degradation on the bucket 250-500Kb. Tolerable degradation on 130-262Kb.

Comparision to Ethereum rate

Per time rate = 3 * 5/12 / 6 * 5/12 = 1.25 / 2.5

Proposal

1/2 is a conservation choice that should not increase network latency beyond key thresholds. Increasing this values can be done latter in response to demand and only requires changes on the execution layer clients.

Closes #18

@4rgon4ut
Copy link
Contributor

@dapplion Looks good!

Full doc to follow

Is there supposed to be a link?

Copy link
Collaborator

@yperbasis yperbasis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe that in that case DATA_GASPRICE_UPDATE_FRACTION should be set to 2225652. Refer to ethereum/EIPs#7154

@yperbasis
Copy link
Collaborator

DATA_GASPRICE_UPDATE_FRACTION

Actually, sorry, 2225652 was the value for 2/4. We need to calculate the correct value for 1/2.

@yperbasis
Copy link
Collaborator

BLOB_GASPRICE_UPDATE_FRACTION should be set to 1112826 so that e(TARGET_BLOB_GAS_PER_BLOCK / BLOB_GASPRICE_UPDATE_FRACTION) ≈ 1.125 still holds.

@4rgon4ut
Copy link
Contributor

1/2 justification from network load perspective

@dapplion dapplion merged commit 512a57a into master Oct 18, 2023
@dapplion dapplion deleted the eip4844-values branch October 18, 2023 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setting EIP-4844 TARGET/MAX_BLOB_GAS_PER_BLOCK
3 participants