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

Update command for running the vm #638

Open
cicr99 opened this issue Aug 16, 2024 · 0 comments
Open

Update command for running the vm #638

cicr99 opened this issue Aug 16, 2024 · 0 comments
Labels
blocked Depends on another to be solved first Difficulty: intermediate Requires some skill but you will get help Duration: under a day Should only take a few hours needs discussion Discuss solution before start implementing it

Comments

@cicr99
Copy link
Contributor

cicr99 commented Aug 16, 2024

Currently the pipeline of the run command, assumes we are dealing with a Cairo Zero program. This needs to be updated to run cairo 1 programs as well.

At the moment this issue is blocked since we need the json output form the cairo file, which is being currently worked on starkware-libs/cairo#5912

Also it remains to be seen if it's worth it to keep the same code for both versions or if we should create a new command altogether.

@cicr99 cicr99 added Difficulty: intermediate Requires some skill but you will get help Duration: under a day Should only take a few hours needs discussion Discuss solution before start implementing it blocked Depends on another to be solved first labels Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Depends on another to be solved first Difficulty: intermediate Requires some skill but you will get help Duration: under a day Should only take a few hours needs discussion Discuss solution before start implementing it
Projects
None yet
Development

No branches or pull requests

1 participant