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

EIP-4844 Implementers' Call #4 #663

Closed
timbeiko opened this issue Nov 14, 2022 · 4 comments
Closed

EIP-4844 Implementers' Call #4 #663

timbeiko opened this issue Nov 14, 2022 · 4 comments

Comments

@timbeiko
Copy link
Contributor

timbeiko commented Nov 14, 2022

@terencechain
Copy link
Contributor

Unfortunately, I won't be able to make it this week due to traveling. Here is the pr/issue I'm looking for feedback on. We don't have to bring them up in the call. Aync feedback is also fine 🙏🏼
ethereum/consensus-specs#3089
ethereum/consensus-specs#3099

@adietrichs
Copy link
Member

Small updates on my PRs again:

@gakonst
Copy link

gakonst commented Nov 15, 2022

Testing / tooling updates

Implementing it this week and will circle back. Apologies for the delay, busy last few weeks.

Our understanding of scope:

  • CLI which takes as inputs:
    • Number of blocks to run experiment for (N)
    • Total data to consume per block (len)
    • Chunk size (chunk-size)
    • Private Key with enough ETH to sign the transactions
    • RPC URL that will not drop the transactions
  • Logic: Loop N times:
    • Generate len random bytes vector
    • Chunk it to len / chunk-size byte vectors with chunk-size length each
    • Create len / chunk-size transactions: Each byte vec is inserted to the calldata field (NOT 4844 blob related)
  • Submit to RPC (or builder to bypass the mempool 128kb limit

@timbeiko
Copy link
Contributor Author

timbeiko commented Nov 19, 2022

Closed in favour of #670

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

No branches or pull requests

4 participants