milvus-docs icon indicating copy to clipboard operation
milvus-docs copied to clipboard

Milvus should be from langchain_milvus.vectorstores

Open jameswnl opened this issue 1 year ago • 3 comments

Without this change, it will fail with

Invalid Milvus URI: ./milvus_demo.db

But even with this change, it still fail with

pymilvus.exceptions.MilvusException: <MilvusException: (code=2, message=Fail connecting to server on unix:/var/folders/r8/vnb_8wm96j151_t9yms3159c0000gn/T/tmpdudd4706_milvus_demo.db.sock, illegal connection params or server unavailable)>

jameswnl avatar Aug 29 '24 06:08 jameswnl

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jameswnl

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

sre-ci-robot avatar Aug 29 '24 06:08 sre-ci-robot

Welcome @jameswnl! It looks like this is your first PR to milvus-io/milvus-docs 🎉

sre-ci-robot avatar Aug 29 '24 06:08 sre-ci-robot

@jameswnl thanks for your feedback .
https://github.com/langchain-ai/langchain-milvus/blob/main/libs/milvus/langchain_milvus/init.py here the outermost layer declares the Milvus/Zilliz class, so there is no need to from langchain_milvus.vectorstores import Milvus I think it's other reasons that cause your issues. So far Milvus Lite only support ubunbu >=20.04, and MacOS, can you provide your OS informations, as well as the langchain_milvus, and pymilvus, and milvus-lite versions

zc277584121 avatar Oct 09 '24 09:10 zc277584121