-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add ksoroban prove
command
#20
Conversation
I scuffed them somehow when doing some reformatting before a commit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good, although I didn't check everything. Most likely, @bbyalcinkaya should approve, but I can also do that if needed.
My only question is whether run_prove
uses the booster or not. I'm guessing that it does not (I may be wrong), so the next question would be if you want to use it.
I was trying to determine if the booster is being used when the proof runner is invoked here, but I haven't been able to so far. I of course would like to make sure that the booster is being used, and planned to ask about it when working on the next PR. |
Since there are only LLVM (without |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good, thank you!
* update runners * update runners
This adds
ksoroban prove
to the CLI:ksoroban prove run
will run the proofs for the test contract in the current working directory.ksoroban prove view
will view the proof id that was saved in the folder specified by--proof-dir
and--id
It's currently limited by the symbolic variables it can generate, notably
Vec
,Map
, andSymbol
are not constrained properly, but all of the integral variables should be ok.