-
Notifications
You must be signed in to change notification settings - Fork 304
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
Vector Search Capabilities... #241
Comments
@VagyokC4 - FWIW you are going to get a The other thing of course is if webdis will pass the parameters correctly passing arrays to Redis as a single parameter is somewhat novel - though I suspect if Webdis is just forwarding the parameters this probably won't be a problem. |
Hi @VagyokC4, I'm not familiar with this feature and haven't used Redis Stack or any of the commercial versions of Redis so it's difficult for me to answer your question. I looked at the docs for The main point of concern for me is the mention of parameters being arrays:
I don't see anything in the If you could give me an example of what the command would look like and specify the types of values being passed in and/or returned, it would really help. I also noticed The main wire format for Webdis inputs and outputs is JSON, and JSON doesn't support binary fields without some kind of encoding. |
Hi @nicolasff So FT.SEARCH idx "*=>[KNN 10 @vec $BLOB]" PARAMS 2 BLOB "\x12\xa9\xf5\x6c" DIALECT 2
|
Hi @nicolasff. How have you been. Hopefully staying safe out there. I wanted to enquire about utilizing the new Vector search from Redis.
Here is the conversation I have with Redis-OM library maintainer, and here are his comments regarding the implementation of the Vector Search capabilities.
redis/redis-om-dotnet#352 (comment)
Just getting your thoughts on the best way to implement (if possible) doing vector searches of Redis via Webdis. It looks like I just need to stringify the current changes to get functionality back to where it was.
Wanted to see if we can get vector searching supported or if it can be supported.
The text was updated successfully, but these errors were encountered: