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

Merged few VM fixes #1820

Merged
merged 16 commits into from
Sep 4, 2024
Merged

Conversation

yuvalsw
Copy link
Collaborator

@yuvalsw yuvalsw commented Aug 8, 2024

TITLE

Merge few fixes

Description

  • Fix Zero segment location.
  • Fixed has_zero_segment naming.
  • Fixed prover input.
  • Fixed version reading when no version is supplied.
  • Added change to changelog.

Checklist

@yuvalsw yuvalsw changed the title Yuval/for sw fix version Merged few VM fixes Aug 8, 2024
vm/src/cairo_run.rs Outdated Show resolved Hide resolved
@yuvalsw yuvalsw force-pushed the yuval/for-sw-fix-version branch from a0cfee5 to 0530a37 Compare August 13, 2024 05:32
@yuvalsw yuvalsw force-pushed the yuval/for-sw-fix-version branch from 1baa9ba to 057bc6f Compare August 15, 2024 09:10
@pefontana pefontana added this pull request to the merge queue Sep 4, 2024
Merged via the queue into lambdaclass:main with commit ed31170 Sep 4, 2024
67 of 70 checks passed
notlesh pushed a commit to Moonsong-Labs/cairo-vm that referenced this pull request Sep 26, 2024
* Fix Zero segment location.

* Fixed has_zero_segment naming

* Fix prover input.

* Fixed version reading when no version is supplied

* Added change to changelog.

* fix test_from_serializable()

* fix panic_impl error

* fix cairo version

* remove outdated test

* Enable ensure-no_std on test

* Add correct test

* Rename tset

* Add comment

---------

Co-authored-by: Alon Titelman <[email protected]>
Co-authored-by: Omri Eshhar <[email protected]>
Co-authored-by: Julián González Calderón <[email protected]>
Co-authored-by: OmriEshhar1 <[email protected]>
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.

7 participants