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

feat(starknet_os): bucket element of 252 bits is felt #4642

Merged
merged 2 commits into from
Mar 4, 2025

Conversation

yoavGrs
Copy link
Contributor

@yoavGrs yoavGrs commented Mar 4, 2025

No description provided.

@reviewable-StarkWare
Copy link

This change is Reviewable

@yoavGrs yoavGrs self-assigned this Mar 4, 2025
@yoavGrs yoavGrs marked this pull request as ready for review March 4, 2025 09:49
Copy link
Collaborator

@dorimedini-starkware dorimedini-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @yoavGrs)

@yoavGrs yoavGrs force-pushed the yoav/snos/hints/compression_hint/pack_usize_in_felt branch from 8e797d5 to 7a22197 Compare March 4, 2025 11:18
@yoavGrs yoavGrs force-pushed the yoav/snos/hints/compression_hint/bucket_element_252 branch from 1797efd to 610d50d Compare March 4, 2025 11:18
@yoavGrs yoavGrs force-pushed the yoav/snos/hints/compression_hint/pack_usize_in_felt branch from 7a22197 to e1e6492 Compare March 4, 2025 12:06
@yoavGrs yoavGrs force-pushed the yoav/snos/hints/compression_hint/bucket_element_252 branch from 610d50d to 03a0683 Compare March 4, 2025 12:06
@yoavGrs yoavGrs force-pushed the yoav/snos/hints/compression_hint/pack_usize_in_felt branch from e1e6492 to b197099 Compare March 4, 2025 12:53
@yoavGrs yoavGrs force-pushed the yoav/snos/hints/compression_hint/bucket_element_252 branch from 03a0683 to 90dd1b3 Compare March 4, 2025 12:53
@yoavGrs yoavGrs changed the base branch from yoav/snos/hints/compression_hint/pack_usize_in_felt to main March 4, 2025 13:29
Copy link

github-actions bot commented Mar 4, 2025

Artifacts upload workflows:

Copy link

github-actions bot commented Mar 4, 2025

Benchmark movements:
tree_computation_flow performance improved 😺
tree_computation_flow time: [34.279 ms 34.354 ms 34.463 ms]
change: [-4.3405% -2.7615% -1.3241%] (p = 0.00 < 0.05)
Performance has improved.
Found 5 outliers among 100 measurements (5.00%)
1 (1.00%) high mild
4 (4.00%) high severe

@yoavGrs yoavGrs added this pull request to the merge queue Mar 4, 2025
Copy link
Collaborator

@dorimedini-starkware dorimedini-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @yoavGrs)

Merged via the queue into main with commit fa86db8 Mar 4, 2025
27 checks passed
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.

3 participants