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

EC_DOUBLE_ASSIGN_NEW_X hint shouldn't appear in issue #1031 #1066

Closed
pefontana opened this issue Apr 26, 2023 · 0 comments · Fixed by #1083
Closed

EC_DOUBLE_ASSIGN_NEW_X hint shouldn't appear in issue #1031 #1066

pefontana opened this issue Apr 26, 2023 · 0 comments · Fixed by #1083
Assignees
Labels
whitelisted-hint Implementation of hint on whitelist directory

Comments

@pefontana
Copy link
Collaborator

EC_DOUBLE_ASSIGN_NEW_X hint shouldn't appear in issue #1031

%{
from starkware.cairo.common.cairo_secp.secp_utils import SECP_P, pack

slope = pack(ids.slope, PRIME)
x = pack(ids.pt.x, PRIME)
y = pack(ids.pt.y, PRIME)

value = new_x = (pow(slope, 2, SECP_P) - 2 * x) % SECP_P
%}

This can happen because:

  • The hint is duplicated with minor string differences in the whitelisted hint list. If this is the case, handle this string match in the cairo-rs HintProcessor.
  • We use a different string from the one in the whitelisted hint list. If this is the case, fix the string match.
@pefontana pefontana moved this to Todo in Starknet Apr 26, 2023
@pefontana pefontana added the whitelisted-hint Implementation of hint on whitelist directory label Apr 26, 2023
@fmoletta fmoletta self-assigned this Apr 27, 2023
@fmoletta fmoletta linked a pull request Apr 27, 2023 that will close this issue
6 tasks
@pefontana pefontana moved this from Todo to In Review in Starknet Apr 27, 2023
@github-project-automation github-project-automation bot moved this from In Review to Done in Starknet Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
whitelisted-hint Implementation of hint on whitelist directory
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants