mirror of
https://github.com/hwchase17/langchain.git
synced 2025-07-07 05:30:39 +00:00
fixed index name error found at redis new vector test (#2311)
This PR fixes a logic error in the Redis VectorStore class Creating a redis vector store `from_texts` creates 1:1 mapping between the object and its respected index, created in the function. The index will index only documents adhering to the `doc:{index_name}` prefix. Calling `add_texts` should use the same prefix, unless stated otherwise in `keys` dictionary, and not create a new random uuid.
This commit is contained in:
parent
3a2782053b
commit
3b75b004fc
@ -69,7 +69,7 @@ class Redis(VectorStore):
|
|||||||
ids = []
|
ids = []
|
||||||
# Check if index exists
|
# Check if index exists
|
||||||
for i, text in enumerate(texts):
|
for i, text in enumerate(texts):
|
||||||
_key = keys[i] if keys else uuid.uuid4().hex
|
_key = keys[i] if keys else self.index_name
|
||||||
key = f"{prefix}:{_key}"
|
key = f"{prefix}:{_key}"
|
||||||
metadata = metadatas[i] if metadatas else {}
|
metadata = metadatas[i] if metadatas else {}
|
||||||
self.client.hset(
|
self.client.hset(
|
||||||
|
Loading…
Reference in New Issue
Block a user