-
Notifications
You must be signed in to change notification settings - Fork 144
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
Pinecone update to OPEA #420
Pinecone update to OPEA #420
Conversation
1d290c7
to
c6d5a96
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there is no need to add nke-10k-2023.pdf
.
197c309
to
c70532b
Compare
@XinyuYe-Intel -> Thank you, removed the unused pdf files from dataprep |
* update chatqna readme and set env script Signed-off-by: letonghan <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * update for comments Signed-off-by: letonghan <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * add consume Signed-off-by: letonghan <[email protected]> * modify details Signed-off-by: letonghan <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * update codegen readme Signed-off-by: letonghan <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * add patch modifications Signed-off-by: letonghan <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * update codegen readme Signed-off-by: letonghan <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * update ui options Signed-off-by: letonghan <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * udpate codetrans readme Signed-off-by: letonghan <[email protected]> * update docsum & searchqna readme Signed-off-by: letonghan <[email protected]> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Signed-off-by: letonghan <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
56c043b
to
256a58b
Compare
Need some help in adding the PINECONE_KEY to the environment variables for the test |
@ZePan110 -> Test is failing as it cannot get the value for PINECONE_KEY, can you please add the variable |
@pallavijaini0525 This variable has been added by #436, please rebase this branch to include the change. |
Signed-off-by: pallavi jaini <[email protected]>
Signed-off-by: pallavi jaini <[email protected]>
Signed-off-by: pallavi jaini <[email protected]>
for more information, see https://pre-commit.ci Signed-off-by: pallavi jaini <[email protected]>
Signed-off-by: pallavi jaini <[email protected]>
…component Signed-off-by: pallavi jaini <[email protected]>
for more information, see https://pre-commit.ci Signed-off-by: pallavi jaini <[email protected]>
Signed-off-by: pallavi jaini <[email protected]>
for more information, see https://pre-commit.ci Signed-off-by: pallavi jaini <[email protected]>
Signed-off-by: pallavi jaini <[email protected]>
256a58b
to
c446e8c
Compare
Signed-off-by: pallavi jaini <[email protected]>
Signed-off-by: pallavi jaini <[email protected]> Signed-off-by: BaoHuiling <[email protected]>
Signed-off-by: pallavi jaini <[email protected]>
Description
Made changes to data prep and retriever components to align with OPEA changes for pinecone DB
Issues
opea-project/GenAIExamples#411
Type of change
List the type of change like below. Please delete options that are not relevant.