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

langchain_ollama: Support keep_alive in embeddings #30251

Merged
merged 3 commits into from
Mar 14, 2025

Conversation

skonto
Copy link
Contributor

@skonto skonto commented Mar 12, 2025

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Mar 12, 2025
Copy link

vercel bot commented Mar 12, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Mar 13, 2025 9:42am

@dosubot dosubot bot added the Ɑ: embeddings Related to text embedding models module label Mar 12, 2025
@skonto
Copy link
Contributor Author

skonto commented Mar 14, 2025

Gentle ping @ccurme

@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Mar 14, 2025
@ccurme ccurme merged commit ac22cde into langchain-ai:master Mar 14, 2025
20 checks passed
andrasfe pushed a commit to andrasfe/langchain that referenced this pull request Mar 18, 2025
- Description: Adds support for keep_alive in Ollama Embeddings see
ollama/ollama#6401.
Builds on top of of
langchain-ai#29296. I have this use
case where I want to keep the embeddings model in cpu forever.
- Dependencies: no deps are being introduced.
- Issue: haven't created an issue yet.
andrasfe pushed a commit to andrasfe/langchain that referenced this pull request Mar 18, 2025
- Description: Adds support for keep_alive in Ollama Embeddings see
ollama/ollama#6401.
Builds on top of of
langchain-ai#29296. I have this use
case where I want to keep the embeddings model in cpu forever.
- Dependencies: no deps are being introduced.
- Issue: haven't created an issue yet.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ɑ: embeddings Related to text embedding models module lgtm PR looks good. Use to confirm that a PR is ready for merging. size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants