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

chore: single integral type #6843

Draft
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

michaeljklein
Copy link
Contributor

Description

Problem*

Resolves #6798

Summary*

Additional Context

Documentation*

Check one:

  • No documentation needed.
  • Documentation included in this PR.
  • [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

Copy link
Contributor

github-actions bot commented Dec 18, 2024

Peak Memory Sample

Program Peak Memory %
keccak256 74.51M -6%
workspace 123.78M 0%
regression_4709 235.23M -45%
ram_blowup_regression 74.55M 100%
private-kernel-tail 167.83M -17%
private-kernel-reset 167.87M -77%
private-kernel-inner 167.84M -43%
parity-root 151.57M -12%

Copy link
Contributor

github-actions bot commented Jan 9, 2025

Execution Memory Report

Program Peak Memory %
keccak256 74.55M 0%
workspace 123.69M 0%
regression_4709 316.60M 0%
ram_blowup_regression 74.59M -86%
rollup-base-public 208.58M -57%
rollup-base-private 207.23M -37%
private-kernel-tail 176.35M -3%
private-kernel-reset 176.95M -30%
private-kernel-inner 179.39M -15%

Copy link
Contributor

github-actions bot commented Jan 9, 2025

Compilation Memory Report

Program Peak Memory %
keccak256 74.55M -6%
workspace 123.78M 0%
regression_4709 316.60M -26%
ram_blowup_regression 74.59M 100%
rollup-base-public 208.58M 100%
rollup-base-private 207.23M 100%
private-kernel-tail 176.35M -15%
private-kernel-reset 176.95M -76%
private-kernel-inner 179.39M -40%

@TomAFrench TomAFrench self-requested a review January 16, 2025 16:33
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.

Combine Field, bool, and () into Type::Integer
2 participants