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

Feature: PIE example + run bootloader helper #5

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

odesenfans
Copy link
Contributor

Problems:

  1. We do not have a working example for Cairo PIEs
  2. It would be convenient for developers to have a helper function to run the bootloader in proof mode easily.

Solutions:

  1. Add an example for how to run a Cairo PIE
  2. Move the cairo_run_bootloader_in_proof_mode from the Stone prover SDK to this repository. This helper is now implemented in a way that makes it easy to reimplement your own version if needed.

Type

  • feature
  • bugfix
  • dev (no functional changes, no API changes)
  • fmt (formatting, renaming)
  • build
  • docs
  • testing

Description

Breaking changes?

  • yes, renamed a public helper function.
  • no

Problems:
1. We do not have a working example for Cairo PIEs
2. It would be convenient for developers to have a helper function to
   run the bootloader in proof mode easily.

Solutions:
1. Add an example for how to run a Cairo PIE
2. Move the `cairo_run_bootloader_in_proof_mode` from the Stone prover
   SDK to this repository. This helper is now implemented in a way that
   makes it easy to reimplement your own version if needed.
@odesenfans odesenfans force-pushed the od/run-bootloader-helper branch from ebec8ad to 5b78320 Compare June 17, 2024 09:50
Copy link
Contributor

@whichqua whichqua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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

Successfully merging this pull request may close these issues.

2 participants