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

Stone Prover Integration (Cairo VM1) #249

Closed
Tracked by #1259
mariari opened this issue Mar 8, 2024 · 2 comments
Closed
Tracked by #1259

Stone Prover Integration (Cairo VM1) #249

mariari opened this issue Mar 8, 2024 · 2 comments
Assignees

Comments

@mariari
Copy link
Member

mariari commented Mar 8, 2024

customer: @mariari
performer: @XuyangSong
deadline: null
estimated: null
started: null
actual: null
completed: null
confirmed: null
dependencies:

  • [ ]

This is an alternative to #248. This is superior in that we can actually pass Resource parameters to contracts. This has some drawbacks discussed in #250.

@mariari mariari changed the title Cairo VM1 Compilation Cairo VM1 Integration and Packaging Mar 8, 2024
@mariari mariari changed the title Cairo VM1 Integration and Packaging Cairo VM1 Integration and Vendoring Mar 8, 2024
@mariari mariari changed the title Cairo VM1 Integration and Vendoring Cairo VM1 Integration Mar 8, 2024
@mariari mariari changed the title Cairo VM1 Integration Stone Prover Integration (Cairo VM1) Mar 8, 2024
@XuyangSong
Copy link
Contributor

XuyangSong commented Mar 14, 2024

The Cairo-vm-1 seems to be doing well with external inputs in cairo-platinum-prover(Winterfell).

  1. run cairo1-run(cairo-vm)
cargo run ../cairo_programs/cairo-1-programs/with_input/array_input_sum.cairo --layout all_cairo --args '2 [1 2 3 4] 0 [9 8]' --trace_file ../cairo_programs/cairo-1-programs/with_input.trace --memory_file ../cairo_programs/cairo-1-programs/with_input.memory --proof_mode
  1. run cairo-platinum-prover(https://github.com/lambdaclass/lambdaworks/tree/main/provers/cairo)
# PROVE
cargo run --release --features=cli,instruments,parallel prove with_input.trace with_input.memory with_input.proof

# VERIFY
cargo run --release --features=cli,instruments,parallel verify with_input.proof

We might not have to deal with the annoying C++ stone-prover. Can you help me double-check it? @CarloModicaPortfolio

@CarloModicaPortfolio
Copy link
Contributor

doube-check passed!

@mariari mariari moved this from Pending to TODO in Skunks Issue Overview Mar 14, 2024
@mariari mariari closed this as completed Mar 18, 2024
@github-project-automation github-project-automation bot moved this from TODO to Done in Skunks Issue Overview Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

3 participants