-
Notifications
You must be signed in to change notification settings - Fork 24
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
feat(papyrus_storage): add sierra version to papyrus cache #2438
base: aviv/get_versioned_casm
Are you sure you want to change the base?
feat(papyrus_storage): add sierra version to papyrus cache #2438
Conversation
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
7f7dbe8
to
3de0987
Compare
865d1dc
to
2fdc7e4
Compare
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.
Reviewed 9 of 9 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @TzahiTaub and @Yoni-Starkware)
a discussion (no related file):
General remark: I have a feeling the size of the sierra version cache should be identical to the size of the sierra contract cache -when will you want a version with the class / a class without the version?
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.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @AvivYossef-starkware and @TzahiTaub)
a discussion (no related file):
Previously, dorimedini-starkware wrote…
General remark: I have a feeling the size of the sierra version cache should be identical to the size of the sierra contract cache -when will you want a version with the class / a class without the version?
Blocking - this PR is not relevant, we're going to have a cache for the Sierra classes.
2fdc7e4
to
3f7ecd2
Compare
3de0987
to
5f9927f
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## aviv/get_versioned_casm #2438 +/- ##
==========================================================
Coverage ? 60.25%
==========================================================
Files ? 180
Lines ? 20780
Branches ? 20780
==========================================================
Hits ? 12522
Misses ? 7426
Partials ? 832 ☔ View full report in Codecov by Sentry. |
f9fb3ea
to
6475cf1
Compare
5f9927f
to
f0084d8
Compare
6475cf1
to
18aa37e
Compare
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.
Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @AvivYossef-starkware and @TzahiTaub)
f0084d8
to
2779f87
Compare
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.
Reviewed 3 of 3 files at r3, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @TzahiTaub)
18aa37e
to
5fe2ded
Compare
5fe2ded
to
4265b9e
Compare
428e056
to
b5d8874
Compare
73d2c81
to
28ce099
Compare
9c10db4
to
3f30869
Compare
ade6fb9
to
d2f5604
Compare
3f30869
to
9c1e654
Compare
d2f5604
to
ade6fb9
Compare
9c1e654
to
4068cd0
Compare
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.
Reviewed 1 of 1 files at r8, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @TzahiTaub)
ade6fb9
to
7763b88
Compare
4068cd0
to
f1edf4a
Compare
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.
Reviewed 1 of 1 files at r9, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @TzahiTaub)
7763b88
to
0999dfe
Compare
f1edf4a
to
a4c372e
Compare
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.
Reviewed 2 of 2 files at r10, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @TzahiTaub)
0999dfe
to
0637840
Compare
a4c372e
to
fd89b79
Compare
0637840
to
447a92d
Compare
fd89b79
to
d7a28ec
Compare
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.
Reviewed 1 of 1 files at r11, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @TzahiTaub)
447a92d
to
ceffe13
Compare
d7a28ec
to
b049066
Compare
ceffe13
to
12d69a6
Compare
b049066
to
a59ff17
Compare
12d69a6
to
1236dca
Compare
a59ff17
to
4215de2
Compare
1236dca
to
4471084
Compare
4215de2
to
d076f8e
Compare
No description provided.