Conversation
ybtsdst
commented
Mar 1, 2024
- update hippo client
- enable hippo client
- add more params
- enable scalar index for filter search cases
- some small improvements
- enable avg latency
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: ybtsdst The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
|
||
| [*.json] | ||
| indent_style = space | ||
| indent_size = 4 No newline at end of file |
There was a problem hiding this comment.
each file should end with a newline, same for other files.
| @@ -0,0 +1,93 @@ | |||
| from transwarp_hippo_api.hippo_client import HippoClient, HippoField | |||
There was a problem hiding this comment.
Should this file be included in this PR? Seems it's just for debugging purpose, we should keep hippo client same as others.
| def _serial_search(self) -> tuple[float, float]: | ||
| def _serial_search(self) -> tuple[float, float, float]: | ||
| """Performance serial tests, search the entire test data once, | ||
| calculate the recall, serial_latency_p99 |
There was a problem hiding this comment.
Maybe also change the docstring? TBH I think the avg latency should be in a seperate PR ;)
XuanYang-cn
left a comment
There was a problem hiding this comment.
@ybtsdst Thanks for contribution!
Hippo clients part LGTM, but this PR carries many unnececcery changes as @zhjwpku suggests. I'll merge this PR if this PR contians changes of hippo client only. Which means the follow change requests:
- remove all the changes about editor and vscode, except
.gitignore - remove the
test.pyin dirhippo/ - remove the new added
avg_latencymetric, better in a different PR