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

error message is confusing for the example on the treadme.md #104

Open
rahul1990gupta opened this issue Sep 14, 2024 · 0 comments
Open

error message is confusing for the example on the treadme.md #104

rahul1990gupta opened this issue Sep 14, 2024 · 0 comments

Comments

@rahul1990gupta
Copy link

Thanks for putting this extension together.

As I was going through the readme page of the extension. I noticed that it was working with 'k= ?', but not with the limit. That should be okay. But the message was saying that either of them will do in a query. So,

  • either we change the message
  • support limit and k both.
sqlite> select rowid, distance from vec_examples where sample_embedding match '[0.890, 0.544, 0.825, 0.961, 0.358, 0.0196, 0.521, 0.175]' and k=1;
2|2.38687372207642
sqlite> 
sqlite> 
sqlite> 
sqlite> select rowid, distance from vec_examples where sample_embedding match '[0.890, 0.544, 0.825, 0.961, 0.358, 0.0196, 0.521, 0.175]' limit 2 ;
Error: in prepare, A LIMIT or 'k = ?' constraint is required on vec0 knn queries. (1)
@rahul1990gupta rahul1990gupta changed the title error message is confusing for the example error message is confusing for the example on the treadme.md Sep 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant