-
Notifications
You must be signed in to change notification settings - Fork 171
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 blockifier #1708
update blockifier #1708
Conversation
a11856f
to
7bddd53
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1708 +/- ##
==========================================
- Coverage 75.51% 75.27% -0.24%
==========================================
Files 90 90
Lines 9631 9562 -69
==========================================
- Hits 7273 7198 -75
- Misses 1828 1830 +2
- Partials 530 534 +4 ☔ View full report in Codecov by Sentry. |
Note: there's are still a few |
3da1647
to
855b063
Compare
708b5f7
to
1363ba0
Compare
We need to pass "L1 data gas consumed" to go side as well. |
and use 0_13 by default
f20edc2
to
f11b72d
Compare
…indEth/juno into rianhughes/0.13.1-blockfier-deps
0d6ae1f breaks TestV1Call for some reason? Tests pass on main, and 97bb768. But not when merged. Failed to read from state: error parsing JSON string for class hash 0x01338d85d3e579f6944ba06c005238d145920afeb32f94e3a1e234d21e1e9292. We should probably pass in a nested list of ints rather than |
Closing in favor of #1728 |
closes #1707
Upgrade the compiler and blockifier dependencies (cairo 2.6.0 / blockifier 0.5.0)
Note: starknet_api also needed to be upgraded to work with the updated cairo-lang-X /blockifier deps