Skip to content

Conversation

medo
Copy link
Contributor

@medo medo commented Sep 21, 2024

Generates embeddings for text/links/pdf bookmarks. This PR does not yet store the embeddings anywhere, instead the embeddings are just being logged.

medo and others added 14 commits September 17, 2024 14:31
Existing version is not working with the upgraded version of drizzle-orm.

I removed the "driver" to the match the new schema of the Config.

Quoting from their Config:
* `driver` - optional param that is responsible for explicitly providing a driver to use when accessing a database
 * *Possible values*: `aws-data-api`, `d1-http`, `expo`, `turso`, `pglite`
 * If you don't use AWS Data API, D1, Turso or Expo - ypu don't need this driver. You can check a driver strategy choice here: https://orm.
@MohamedBassem MohamedBassem merged commit e96ff9e into karakeep-app:rag Dec 29, 2024
MohamedBassem added a commit that referenced this pull request Dec 29, 2024
* support embeddings generation in inference.ts

(cherry picked from commit 9ae8773)

* make AI worker generate embeddings for text bookmark

* make AI worker generate embeddings for text bookmark

* fix unintentional change -- inference image model

* support embeddings for PDF bookmarks

* Upgrade drizzle-kit

Existing version is not working with the upgraded version of drizzle-orm.

I removed the "driver" to the match the new schema of the Config.

Quoting from their Config:
* `driver` - optional param that is responsible for explicitly providing a driver to use when accessing a database
 * *Possible values*: `aws-data-api`, `d1-http`, `expo`, `turso`, `pglite`
 * If you don't use AWS Data API, D1, Turso or Expo - ypu don't need this driver. You can check a driver strategy choice here: https://orm.

* fix formatting and lint

* add comments about truncate content

* Revert "Upgrade drizzle-kit"

This reverts commit 08a02c8.

* revert keep alive field in Ollama

* change the interface to accept multiple inputs

* docs

---------

Co-authored-by: Mohamed Bassem <me@mbassem.com>
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

Successfully merging this pull request may close these issues.

2 participants