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

NewHint#22 shouldn't appear in issue #1031 #1068

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

NewHint#22 shouldn't appear in issue #1031 #1068

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

Comments

@pefontana
Copy link
Collaborator

NewHint#22 shouldn't appear in issue #1031

%{
from starkware.cairo.common.cairo_secp.secp_utils import pack
x = pack(ids.x, PRIME) % 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
@fmoletta fmoletta linked a pull request Apr 27, 2023 that will close this issue
6 tasks
@github-project-automation github-project-automation bot moved this from Todo 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