Skip to content

Initial LLM integration#101

Merged
sgreenbury merged 42 commits intomainfrom
91-llm
Apr 2, 2025
Merged

Initial LLM integration#101
sgreenbury merged 42 commits intomainfrom
91-llm

Conversation

@sgreenbury
Copy link
Contributor

@sgreenbury sgreenbury commented Mar 25, 2025

Closes #91.

This PR:

  • Adds popgetter-llm crate (LLM crate) providing search with text embeddings and DataRequestSpec with langchain
  • Integrates the LLM crate with the CLI (optionally installable with llm feature)

Next steps:

  • Write instructions for: init, query, ...

@sgreenbury sgreenbury marked this pull request as ready for review March 30, 2025 21:31
@sgreenbury
Copy link
Contributor Author

sgreenbury commented Mar 31, 2025

Currently tests passing locally but not on the runner - not clear the reason. Fixed by 79cb65e (compilation memory size appears to have been causing process to be killed).

@sgreenbury sgreenbury merged commit a19258a into main Apr 2, 2025
12 checks passed
@sgreenbury sgreenbury deleted the 91-llm branch April 2, 2025 19:43
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.

Add LLM crate and embeddings

1 participant