Skip to content

Commit

Permalink
Make the bulk-fetch for reindexCardEmbeddings also ask for the card_id.
Browse files Browse the repository at this point in the history
Hopefully this will fix #691. I believe currently the cardsInfo is a map of {'':<last-embedding>} because we don't tell qdrant to include the card_ID in payload.

It's kind of hard to debug this with long cycle times, and if that's true, this optimization NEVER worked. (Or maybe Qdrant changed its behavior and previously returned all payload fields and now listens to the config)?
  • Loading branch information
jkomoros committed Apr 13, 2024
1 parent 886ad2f commit 459ed50
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion functions/src/embeddings.ts
Original file line number Diff line number Diff line change
Expand Up @@ -481,7 +481,10 @@ export const reindexCardEmbeddings = async () : Promise<void> => {
]
},
with_payload: {
include: [PAYLOAD_CONTENT_KEY]
include: [
PAYLOAD_CONTENT_KEY,
PAYLOAD_CARD_ID_KEY
]
}
});

Expand Down

0 comments on commit 459ed50

Please sign in to comment.