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

Updated Pinecone API Key Setup instructions for clarity #313

Merged
merged 1 commit into from
Apr 10, 2023

Conversation

DawgZter
Copy link
Contributor

@DawgZter DawgZter commented Apr 6, 2023

Minor improvement in sentence clarity for Pinecone setup instructions.

Minor improvement in sentence clarity for Pinecone setup instructions.
@DawgZter DawgZter changed the title Update Pinecone API Key Setup instructions for clarity Updated Pinecone API Key Setup instructions for clarity Apr 6, 2023
@DawgZter
Copy link
Contributor Author

DawgZter commented Apr 8, 2023

@Torantulino Let me know what else you need from me :)

@nponeccop
Copy link
Contributor

nponeccop commented Apr 8, 2023

Well, I think we can do better, especially that ChatGPT is good at improving the documentation. What we really need here is to add more details about how it works.

Pinecone is an SVM-like vector database that allows Auto-GPT to store chat history indexed by the vectors generated by the OpenAI text embeddings model, thus allowing Auto-GPT to "remember" past messages using a fuzzy match. This partially solves the problem of having a limited context memory.

I'm not saying this is much better (especially that I'm not confident in my grammar) but the original PR brings very little value IMO. Why the Pinecone is not optional? That isn't clear either.

@nponeccop nponeccop mentioned this pull request Apr 10, 2023
1 task
@Torantulino Torantulino merged commit 977d84b into Significant-Gravitas:master Apr 10, 2023
waynehamadi added a commit that referenced this pull request Sep 5, 2023
Signed-off-by: Merwane Hamadi <merwanehamadi@gmail.com>
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.

3 participants