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

Pinecone update to OPEA #420

Merged
merged 10 commits into from
Aug 12, 2024
Merged

Conversation

pallavijaini0525
Copy link
Contributor

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.

  • Others (enhancement, documentation, validation, etc.)

Copy link
Collaborator

@XinyuYe-Intel XinyuYe-Intel left a 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.

@pallavijaini0525
Copy link
Contributor Author

@XinyuYe-Intel -> Thank you, removed the unused pdf files from dataprep

lkk12014402 pushed a commit that referenced this pull request Aug 8, 2024
* 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>
@pallavijaini0525 pallavijaini0525 force-pushed the pinecone_update branch 2 times, most recently from 56c043b to 256a58b Compare August 8, 2024 21:41
@pallavijaini0525
Copy link
Contributor Author

Need some help in adding the PINECONE_KEY to the environment variables for the test

@pallavijaini0525
Copy link
Contributor Author

@ZePan110 -> Test is failing as it cannot get the value for PINECONE_KEY, can you please add the variable

@ZePan110
Copy link
Collaborator

@pallavijaini0525 This variable has been added by #436, please rebase this branch to include the change.

@chensuyue chensuyue merged commit 7c9f77b into opea-project:main Aug 12, 2024
9 checks passed
@pallavijaini0525 pallavijaini0525 deleted the pinecone_update branch August 12, 2024 16:51
BaoHuiling pushed a commit to siddhivelankar23/GenAIComps that referenced this pull request Aug 15, 2024
BaoHuiling pushed a commit to siddhivelankar23/GenAIComps that referenced this pull request Aug 15, 2024
Signed-off-by: pallavi jaini <[email protected]>
Signed-off-by: BaoHuiling <[email protected]>
sharanshirodkar7 pushed a commit to predictionguard/pg-GenAIComps that referenced this pull request Sep 3, 2024
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.

4 participants