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.

Sign up for GitHub

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

Jump to bottom

Remove/Replace validateBlockForSubmission #11440

Open
Tracked by #9456
spypsy opened this issue Jan 23, 2025 · 0 comments
Open
Tracked by #9456

Remove/Replace validateBlockForSubmission #11440

spypsy opened this issue Jan 23, 2025 · 0 comments

Comments

@spypsy
Copy link
Member

spypsy commented Jan 23, 2025

No description provided.

This was referenced Jan 23, 2025
Open
Merged
spypsy added a commit that referenced this issue Jan 24, 2025
@spypsy @spalladino
fix: use simulation to estimate gas used (#11211)
63776f0 
Relevant to #10991 but not 'Fixes' because anvil doesn't support
`eth_simulateV1` so we can't yet remove all time-based inputs in L1

Fixes #11341 

- We now try to `simulate` the 'propose' & 'proposeAndClaim'
transactions, using the appropriate timestamp, instead of using
hardcoded 12M gas.
- Also some updates to logging where we 
  1. Try to decode errors from our Rollup ABI
2. truncate large Hexes but still include them in error data so txs can
be investigated
- Update slot duration to 36 for sepolia deployments

Follow-up from this: #11390 #11440

---------

Co-authored-by: Santiago Palladino <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@spypsy