-
Notifications
You must be signed in to change notification settings - Fork 76
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: New curp conflict pool #675
Conversation
bca6be2
to
51753e3
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.
Please add some test cases for this new sp and ucp.
7065fbd
to
2fea605
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #675 +/- ##
==========================================
+ Coverage 75.45% 75.62% +0.17%
==========================================
Files 176 180 +4
Lines 26515 26938 +423
Branches 26515 26938 +423
==========================================
+ Hits 20006 20371 +365
- Misses 5297 5346 +49
- Partials 1212 1221 +9 ☔ View full report in Codecov by Sentry. |
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.
LGTM. Please squash these commits.
Signed-off-by: bsbds <[email protected]>
a13ebad
to
c028750
Compare
Signed-off-by: bsbds <[email protected]>
Please briefly answer these questions:
what problem are you trying to solve? (or if there's no problem, what's the motivation for this change?)
First PR of refactor: enable new sp/ucp implementation #668.
Implements new speculative pool and uncommitted pool in curp.
what changes does this pull request make?
are there any non-obvious implications of these changes? (does it break compatibility with previous versions, etc)