[PR #41] [MERGED] Add ability to choose other models #74

Closed
opened 2026-02-16 05:15:18 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langchain-extract/pull/41
Author: @ccurme
Created: 3/18/2024
Status: Merged
Merged: 3/19/2024
Merged by: @ccurme

Base: mainHead: cc/select_models


📝 Commits (10+)

📊 Changes

13 files changed (+260 additions, -24 deletions)

View changed files

📝 backend/poetry.lock (+136 -2)
📝 backend/pyproject.toml (+1 -0)
backend/server/api/configurables.py (+20 -0)
📝 backend/server/api/extract.py (+3 -2)
📝 backend/server/api/suggest.py (+1 -1)
📝 backend/server/extraction_runnable.py (+7 -6)
📝 backend/server/main.py (+2 -1)
backend/server/models.py (+57 -0)
📝 backend/server/retrieval.py (+3 -1)
📝 backend/server/settings.py (+0 -11)
backend/tests/unit_tests/api/test_api_configurables.py (+15 -0)
📝 backend/tests/unit_tests/api/test_api_extract.py (+13 -0)
📝 backend/tests/unit_tests/conftest.py (+2 -0)

📄 Description

https://github.com/langchain-ai/langchain-extract/issues/37

Here we add an argument to ExtractionRequest to specify the underlying chat model.

Alternatively we could add this as a property of an extractor, but I kept them separate so that users could easily experiment with different models for the same underlying schema.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/langchain-extract/pull/41 **Author:** [@ccurme](https://github.com/ccurme) **Created:** 3/18/2024 **Status:** ✅ Merged **Merged:** 3/19/2024 **Merged by:** [@ccurme](https://github.com/ccurme) **Base:** `main` ← **Head:** `cc/select_models` --- ### 📝 Commits (10+) - [`2fbe9ea`](https://github.com/langchain-ai/langchain-extract/commit/2fbe9eacc362e6aa8dcc40d629fa9aaab78fa049) support multiple models - [`4a0d20b`](https://github.com/langchain-ai/langchain-extract/commit/4a0d20bab689fa2eed02b2a3c63e326389521b99) change to literal - [`71b519b`](https://github.com/langchain-ai/langchain-extract/commit/71b519b77d9488d56cd548f3fdeaa5df7124df0a) add api endpoint - [`f995f86`](https://github.com/langchain-ai/langchain-extract/commit/f995f86335261807372c6d96927b2a927c31a048) add test - [`69bf7bd`](https://github.com/langchain-ai/langchain-extract/commit/69bf7bdfb0ddcb62d5790180659ae982a9c15471) fix bug with retrieval implementation - [`abc2ec7`](https://github.com/langchain-ai/langchain-extract/commit/abc2ec7c2273fdca4fc7cf46bd87aba22fd0eb0b) format - [`1937f47`](https://github.com/langchain-ai/langchain-extract/commit/1937f4750802b75673d3ae974ff483e2681ce8e6) merge main - [`0e6b188`](https://github.com/langchain-ai/langchain-extract/commit/0e6b188b93feeb39290425fad23e93c008465b7e) change to configurables - [`1bea75e`](https://github.com/langchain-ai/langchain-extract/commit/1bea75e38b496c4f37c3f69e41f04fc707243aab) add fireworks to dependencies - [`53edada`](https://github.com/langchain-ai/langchain-extract/commit/53edada21f0997afbcab81b826c45a18dd4ca2b6) add fireworks ### 📊 Changes **13 files changed** (+260 additions, -24 deletions) <details> <summary>View changed files</summary> 📝 `backend/poetry.lock` (+136 -2) 📝 `backend/pyproject.toml` (+1 -0) ➕ `backend/server/api/configurables.py` (+20 -0) 📝 `backend/server/api/extract.py` (+3 -2) 📝 `backend/server/api/suggest.py` (+1 -1) 📝 `backend/server/extraction_runnable.py` (+7 -6) 📝 `backend/server/main.py` (+2 -1) ➕ `backend/server/models.py` (+57 -0) 📝 `backend/server/retrieval.py` (+3 -1) 📝 `backend/server/settings.py` (+0 -11) ➕ `backend/tests/unit_tests/api/test_api_configurables.py` (+15 -0) 📝 `backend/tests/unit_tests/api/test_api_extract.py` (+13 -0) 📝 `backend/tests/unit_tests/conftest.py` (+2 -0) </details> ### 📄 Description https://github.com/langchain-ai/langchain-extract/issues/37 Here we add an argument to ExtractionRequest to specify the underlying chat model. Alternatively we could add this as a property of an extractor, but I kept them separate so that users could easily experiment with different models for the same underlying schema. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 05:15:18 -05:00
yindo closed this issue 2026-02-16 05:15:18 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/langchain-extract#74