-
Notifications
You must be signed in to change notification settings - Fork 325
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
Milvus Client #1374
Milvus Client #1374
Conversation
fixes #1380 |
00fb70b
to
db6c2ef
Compare
please squash your commit record into 1 to make the commit history easier to understand. |
66f5344
to
c331cb4
Compare
@filip-halt could you please solve the code style issues, otherwise this can't be merged automatically. |
94e9c4f
to
b8e4764
Compare
b8e4764
to
d9c217f
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: filip-halt, longjiquan The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
d9c217f
to
b25b389
Compare
29bf69b
to
adb07ac
Compare
adb07ac
to
8d7c399
Compare
Signed-off-by: Filip Haltmayer <[email protected]>
8d7c399
to
d07f2ee
Compare
/lgtm |
- Fix incorrect dim error msg (milvus-io#1394) - Include MilvusClient (milvus-io#1374) Co-Authored-by: Filip Haltmayer <[email protected]> Signed-off-by: yangxuan <[email protected]>
- Fix incorrect dim error msg (#1394) - Include MilvusClient (#1374) Signed-off-by: yangxuan <[email protected]> Co-authored-by: Filip Haltmayer <[email protected]> Co-authored-by: Filip Haltmayer <[email protected]>
Looking to add a wrapper around Pymilvus for easier usage.