-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
fix: add crud numpy indexer #1550
Conversation
This PR closes: #1549 |
1 similar comment
This PR closes: #1549 |
Latency summaryCurrent PR yields:
Breakdown
Backed by latency-tracking. Further commits will update this comment. |
8159664
to
7ec0e26
Compare
Codecov Report
@@ Coverage Diff @@
## master #1550 +/- ##
==========================================
+ Coverage 84.37% 84.67% +0.29%
==========================================
Files 108 109 +1
Lines 6311 6465 +154
==========================================
+ Hits 5325 5474 +149
- Misses 986 991 +5
Continue to review full report at Codecov.
|
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.
i see some of redudancy in CRUDBase and Crud, but given this is a temp solution, good to go
No description provided.