feat: add u256 -> Felt scale down procedure#2320
Merged
partylikeits1983 merged 31 commits intoagglayerfrom Jan 22, 2026
Merged
Conversation
partylikeits1983
added
no changelog
partylikeits1983
changed the base branch from
agglayer
to
ajl-claim-note-follow-up
Contributor
|
I think this was merged into the |
Contributor
Author
|
An accidental merge occurred on this branch to |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds functionality which scales a
U256value to a nativeFeltvalue using non-deterministic inputs from theAdviceProvider.TODO: The math works, but need to think through the best way to create an event which is called inside
scale_u256_to_native_amountto call outside to the host to compute the division and push to theAdviceStack.Resolves point 2.2 here: #2237
Builds on top of: #2270
Context: #2011 (comment)