Skip to content
This repository has been archived by the owner on Mar 25, 2024. It is now read-only.

Pubdata compression #23

Merged
merged 13 commits into from
Sep 28, 2023
Merged

Conversation

koloz193
Copy link
Contributor

@koloz193 koloz193 commented Sep 25, 2023

What ❔

Edit the compression algorithm used for state diffs. The prior one was used to make sure the system worked with the old format but this upgrade will allow us to have ~70% gain in data compression.

Change how state diffs are processed at the end of the batch.

  • L1Messenger is updated to parse out the new format/values
  • Compressor is updated to properly compare the encoded state diffs vs the new compressed state diffs

Why ❔

While the current iteration of compression works, we can achieve further compression with this new algorithm bringing down the amount of data needed and thus costs due to pubdata publishing.

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.

@StanislavBreadless StanislavBreadless merged commit 0fd8456 into boojum-integration Sep 28, 2023
1 check passed
@StanislavBreadless StanislavBreadless deleted the pubdata-compression branch September 28, 2023 15:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants