Langchainvdms update #1303
Closed
Langchainvdms update #1303
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Lacewell, Chaunte W <[email protected]>
…#1290) * Fix telemetry connection issue when disabling telemetry - use ENABLE_OPEA_TELEMETRY to control whether to enable open telemetry, default false - fix the issue that logs always show telemetry connection error with each request when telemetry is disabled - ban the above error propagation to microservices when telemetry is disabled Signed-off-by: Spycsh <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Fix ut failure where required the flag to be on * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Signed-off-by: Spycsh <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Signed-off-by: Lacewell, Chaunte W <[email protected]>
Signed-off-by: Lacewell, Chaunte W <[email protected]>
Signed-off-by: Lacewell, Chaunte W <[email protected]>
for more information, see https://pre-commit.ci Signed-off-by: Lacewell, Chaunte W <[email protected]>
cwlacewe
force-pushed
the
langchainvdms_update
branch
from
ceb8509 to
c27de43
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
|
…opea-project#1298) * Refine CLI embedding microservice using dependency Signed-off-by: lvliang-intel <[email protected]> Signed-off-by: Lacewell, Chaunte W <[email protected]>
1. set default session_id for react_langchain strategy, because the langchain version upgrade. 2. fix request message format Signed-off-by: Lacewell, Chaunte W <[email protected]>
Signed-off-by: lvliang-intel <[email protected]> Co-authored-by: Letong Han <[email protected]> Signed-off-by: Lacewell, Chaunte W <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Liang Lv <[email protected]> Signed-off-by: Lacewell, Chaunte W <[email protected]>
* Refine dataprep Milvus CI Signed-off-by: letonghan <[email protected]> Signed-off-by: Lacewell, Chaunte W <[email protected]>
…t#1292) * Change neo4j Bolt default PORT from 7687 to -configured the port in neo4j compose.yaml to use variable value -made all corresponding changes in neo4j dataprep and retriever components and test scripts to use env variable instead of default port value. Signed-off-by: rbrugaro <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * missing positional arg in milvus dataprep Signed-off-by: rbrugaro <[email protected]> * remove redundance in stop_docker Signed-off-by: rbrugaro <[email protected]> * resolve retriever to neo4j connectivity issue bad URL Signed-off-by: rbrugaro <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * set neo4j ports to neo4j defaults and fix environment variables in READMEs Signed-off-by: rbrugaro <[email protected]> --------- Signed-off-by: rbrugaro <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Liang Lv <[email protected]> Signed-off-by: Lacewell, Chaunte W <[email protected]>
…nAIComps into langchainvdms_update
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WIP - PENDING LANGCHAIN MERGE
Description
VDMS integration in LangChain recently updated. This PR adds the necessary changes prior to previous version being deprecated.
Issues
n/a
Type of change
Dependencies
langchain-vdms
Tests
Ran provided tests without any errors: