Skip to content

fix: cast vector_distance to float in Redis search#4377

Merged
whysosaket merged 1 commit intomainfrom
fix/redis-vector-distance-float-cast
Mar 17, 2026
Merged

fix: cast vector_distance to float in Redis search#4377
whysosaket merged 1 commit intomainfrom
fix/redis-vector-distance-float-cast

Conversation

@utkarsh240799
Copy link
Copy Markdown
Contributor

@utkarsh240799 utkarsh240799 commented Mar 17, 2026 •
edited
Loading

Summary

Test plan

  • Verify memory.search() with Redis vector store and a threshold parameter no longer raises TypeError
  • Confirm search results return correct float scores

@utkarsh240799 @claude
fix: cast vector_distance to float in Redis search
5050658 
redisvl returns vector_distance as a string, causing TypeError when
comparing with float threshold values in memory.search().

Fixes #4294

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@whysosaket whysosaket merged commit 8230a5d into main Mar 17, 2026
8 checks passed
@whysosaket whysosaket deleted the fix/redis-vector-distance-float-cast branch March 17, 2026 11:04
This was referenced Mar 17, 2026
Closed
Closed
Closed
jamebobob pushed a commit to jamebobob/mem0-vigil-recall that referenced this pull request Mar 29, 2026
@utkarsh240799 @claude
fix: cast vector_distance to float in Redis search (mem0ai#4377)
cf120ea 
Co-authored-by: utkarsh240799 <utkarsh240799@users.noreply.github.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@whysosaket whysosaket whysosaket approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Redis vector_distance returned as string causes threshold search to fail

2 participants

@utkarsh240799 @whysosaket