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

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

Conversation

Hesse
Copy link

@Hesse Hesse commented Apr 22, 2023

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.

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.
@fabianhug
Copy link

+1

@tdimino
Copy link

tdimino commented May 8, 2023

Include this sample image in the README.md as well:

image

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.

None yet

3 participants