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

Update wasm lane 5. #5055

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

mpapierski
Copy link
Collaborator

This PR adjusts 5th lane in size, and in gas limit.

  • 65k will fit more than half of the wasm sessions used in the mainnet to date
  • 3CSPR will accommodate a Wasm with at least one transfer (2.5CSPR) with some extra code.

See also mainnet data analysis here: https://gist.github.com/mpapierski/1b7a603bde1d4fe99c836ef9b68b3d0e#file-data_with_imports-csv

This should fit a single wasm transfer with some overhead (~2.6CSPR) and
also the size is increased to 2^16. Based on mainnet data this should
fit more than half of the wasm sessions used.
@devendran-m devendran-m added the release blocker PR to be merged before releasing label Jan 9, 2025
@mpapierski
Copy link
Collaborator Author

bors r+

casperlabs-bors-ng bot added a commit that referenced this pull request Jan 10, 2025
5055: Update wasm lane 5. r=mpapierski a=mpapierski

This PR adjusts 5th lane in size, and in gas limit.

- 65k will fit more than half of the wasm sessions used in the mainnet to date 
- 3CSPR will accommodate a Wasm with at least one transfer (2.5CSPR) with some extra code.

See also mainnet data analysis here: https://gist.github.com/mpapierski/1b7a603bde1d4fe99c836ef9b68b3d0e#file-data_with_imports-csv

Co-authored-by: Michał Papierski <michal@casper.network>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release blocker PR to be merged before releasing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants