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

Vector DB integration inside of ChatGPT Agent #264

Merged

Conversation

HHousen
Copy link
Contributor

@HHousen HHousen commented Jun 30, 2023

No description provided.

@linear
Copy link

linear bot commented Jun 30, 2023

@HHousen HHousen force-pushed the hayden/epd-310-vector-db-integration-inside-of-chatgpt-agent branch from a353664 to 3ea8516 Compare June 30, 2023 23:49
Copy link
Collaborator

@Kian1354 Kian1354 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

overall lgtm, left some comments/questions

vocode/streaming/models/vector_db.py Outdated Show resolved Hide resolved
vocode/streaming/vector_db/pinecone.py Outdated Show resolved Hide resolved
vocode/streaming/vector_db/pinecone.py Outdated Show resolved Hide resolved
self.vectorstore = Pinecone(index, self.embeddings.embed_query, "text")

async def add_texts(self, texts, **kwargs):
func = partial(self.vectorstore.add_texts, texts, **kwargs)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does pinecone not support async at all?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, it does not support async.

vocode/streaming/agent/chat_gpt_agent.py Outdated Show resolved Hide resolved
@HHousen HHousen marked this pull request as ready for review July 3, 2023 20:49
@HHousen HHousen merged commit c255dd3 into main Jul 3, 2023
KShah707 pushed a commit to KShah707/vocode-python that referenced this pull request Jul 5, 2023
* Add pinecone integration with ChatGPT agent

* Make top_k default to 3

* Use getenv

* Remove reassigned import

* Add vector db sender instead of sending as bot

* Fix mypy
@rjheeta rjheeta mentioned this pull request Jul 30, 2023
m5a0r7 pushed a commit to m5a0r7/vocode-python that referenced this pull request Oct 19, 2023
* Add pinecone integration with ChatGPT agent

* Make top_k default to 3

* Use getenv

* Remove reassigned import

* Add vector db sender instead of sending as bot

* Fix mypy
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.

2 participants