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

[blocked] Double the amount of bootloader tx slots and memory #647

Closed
wants to merge 1 commit into from

Conversation

ly0va
Copy link
Member

@ly0va ly0va commented Dec 8, 2023

What ❔

Double the amount of bootloader tx slots and memory.
Only merge after matter-labs/era-contracts#112 and matter-labs/era-system-contracts#85 are merged into their respective dev branches and then into this branch.

Why ❔

With pubdata compression we can now accommodate larger L1 batches.

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted via zk fmt and zk lint.
  • Spellcheck has been run via cargo spellcheck --cfg=./spellcheck/era.cfg --code 1.

Copy link
Contributor

github-actions bot commented Dec 8, 2023

Detected VM performance changes

Benchmark name Difference in runtime
write_and_decode -95.4%
event_spam -92.3%
access_memory -95.5%
deploy_simple_contract -10.6%
slot_hash_collision -95.4%
decode_shl_sub -95.2%
call_far -71.3%
finish_eventful_frames -84.0%

@popzxc
Copy link
Member

popzxc commented Feb 7, 2024

@ly0va is this PR still relevant? If not, could you please close it?

@ly0va ly0va closed this Feb 7, 2024
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.

2 participants