community: Bug Fix, in Neo4j VectorStore when having multiple indexes the sort is not working and the store that returned is random #17396
Conversation
… store that returned is random. The following small fix resolves the issue.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
dosubot
bot
added
size:XS
|
cc @tomasonjo |
|
@ehude looks ok. Can you please add a test. Can copy from here: #16505 or you can check the test I did in JS version: https://github.com/langchain-ai/langchainjs/pull/4390/files |
dosubot
bot
added
size:M
|
@tomasonjo thanks for the info. |
|
You need to create two indexes, but then use |
|
make more sense indeed :) |
|
Awesome, thank you! |
|
Glad I could help :) |
|
@ehude can you check the lint error, so that we can get this in? Thanks |
|
@tomasonjo Sure, will fix it tomorrow |
|
@tomasonjo |
seems like formatting is missing. If you run |
|
done |
|
Thanks @ehude . @baskaryan LGTM! |
|
Can you move it from unit tests to integration tests, and then it should pass all checks :) |
|
Thanks |
dosubot
bot
added
the
lgtm
…the sort is not working and the store that returned is random (langchain-ai#17396) Bug fix: when having multiple indexes the sort is not working and the store that returned is random. The following small fix resolves the issue.
…the sort is not working and the store that returned is random (langchain-ai#17396) Bug fix: when having multiple indexes the sort is not working and the store that returned is random. The following small fix resolves the issue.
…the sort is not working and the store that returned is random (langchain-ai#17396) Bug fix: when having multiple indexes the sort is not working and the store that returned is random. The following small fix resolves the issue.
Bug fix: when having multiple indexes the sort is not working and the store that returned is random.
The following small fix resolves the issue.