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.

Sign up for GitHub

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

Jump to bottom

Update README.md - add the 1536 dimension number at the start so you don't get bit later. #199

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update README.md - add the 1536 dimension number at the start so you don't get bit later. #199

wants to merge 1 commit into from

Commits on Apr 22, 2023

  1. Update README.md 

    A just a small edit on the getting started steps. I went ahead and blindly created an index in pinecone not knowing what I should set the dimensions to, so I arbitrarily put in a number, which later came back to bite me because it wasn't in line with OpenAI's 1536. I only found this out after it failed ingestion. If I had known to start with 1536 from the start, I could have avoided this. 

I know you mention this as a debug step in the bottom, but placing it right at the start could avoid this issue in the first place.
    @Hesse
    Hesse authored Apr 22, 2023
    Configuration menu
    Copy the full SHA
    e865855 View commit details
    Browse the repository at this point in the history