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

Check if we should skip dynamic layout params when generating air public inputs #1649

Closed
fmoletta opened this issue Mar 5, 2024 · 0 comments · Fixed by #1716
Closed

Check if we should skip dynamic layout params when generating air public inputs #1649

fmoletta opened this issue Mar 5, 2024 · 0 comments · Fixed by #1716
Labels
question Further information is requested

Comments

@fmoletta
Copy link
Contributor

fmoletta commented Mar 5, 2024

Looking at the cairo_lang code, when the public inputs are generated (reference) the layout params are set to None by default even if the layout is dynamic. If we run a program using a dynamic layout and output the public inputs using cairo-lang we can see that the dynamic_params field is null. In cairo-vm we write the layout params when the layout is dynamic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant