Skip to content
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

fix(vector): Update query_rewriter to fix dotproduct and cosine query conversion #9083

Merged
merged 6 commits into from
May 15, 2024

Conversation

@rderbier rderbier requested a review from a team as a code owner May 7, 2024 06:42
@dgraph-bot dgraph-bot added area/graphql Issues related to GraphQL support on Dgraph. area/core internal mechanisms go Pull requests that update Go code labels May 7, 2024
@rderbier
Copy link
Contributor Author

rderbier commented May 7, 2024

Not sure why but if the $search_vector is not in (), we get an error.
Updating the math function solved the issue.

harshil-goel
harshil-goel previously approved these changes May 7, 2024
@harshil-goel harshil-goel changed the title Update query_rewriter.go fix(vector): Update query_rewriter to fix dotproduct and cosine query conversion May 15, 2024
@harshil-goel harshil-goel merged commit 207583d into main May 15, 2024
12 checks passed
@harshil-goel harshil-goel deleted the raphael/distance branch May 15, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/core internal mechanisms area/graphql Issues related to GraphQL support on Dgraph. go Pull requests that update Go code
Development

Successfully merging this pull request may close these issues.

4 participants