OoriData / OgbujiPT

Client-side toolkit for using large language models, including where self-hosted
Apache License 2.0
101 stars 8 forks source link

ogbujipt.embedding.pgvector.PGVectorHelper.insert_many not working #54

Closed chimezie closed 9 months ago

chimezie commented 9 months ago

If you pass it the expected list of tuples of the form:

It raises the exception:

asyncpg.exceptions._base.InterfaceError: the server expects 5 arguments for this query, 6 were passed

Looking at INSERT_DOCS in that module, it is not expecting the 'permission' item in the tuple. It looks like recent changes to remove the permission from the embedding table didn't update this method and introduced a regression. It would probably be good to have a unit test for this function to prevent such changes from breaking it.

choccccy commented 9 months ago

included a unit test and merged it into your branch as of commit c497918

choccccy commented 9 months ago

this should be solved as of the merge from Pgvector bigdata insert and search and released in version 0.7.0