Skip to content

Update the LlamaIndex notebook to current SDKs - #624

Merged
nicoalba10 merged 3 commits into
mainfrom
update-llamaindex-notebook
Oct 8, 2026
Merged

nicoalba10 merged 3 commits into
mainfrom
update-llamaindex-notebook

Conversation

@nicoalba10

@nicoalba10 nicoalba10 commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Ports the tested code from the LlamaIndex integration guide so the notebook matches the page step by step. It was pinned to pinecone-client[grpc]==3.0.0 and llama-index==0.9.34 and used imports that no longer exist.

  • Pins pinecone==9.1.0 (the highest version llama-index-vector-stores-pinecone==0.9.0 allows, since it requires pinecone<10), llama-index==0.14.25, and the split llama-index-* packages.
  • Uses text-embedding-3-small for embeddings and gpt-4o-mini for generation and evaluation.
  • Adds the out-of-scope evaluation example from the page and a cleanup cell that deletes the index.
  • Points the Colab and nbviewer badges at main.

Ran end to end on Python 3.12: 46 vectors upserted, the relevancy check passes for the in-scope query and fails for the out-of-scope one.


Note

Low Risk
Documentation-only notebook update with no production code or auth changes; risk is limited to example correctness and pinned dependency versions.

Overview
Brings using-llamaindex-with-pinecone.ipynb in line with the LlamaIndex integration guide so it runs on current Pinecone and LlamaIndex packages.

Dependency and API refresh: Replaces legacy pins (pinecone-client[grpc]==3.0.0, monolithic llama-index==0.9.34) with pinecone==9.1.0, llama-index==0.14.25, and split packages (llama-index-vector-stores-pinecone, llama-index-readers-file, etc.), and updates imports to the llama_index.core / integration module layout (PineconeGRPC, PineconeVectorStore, IngestionPipeline, and related APIs).

Models and flow: Uses text-embedding-3-small for splitting/embeddings and gpt-4o-mini for RAG answers and RelevancyEvaluator checks. Adds the guide’s out-of-scope query/eval example and a cleanup cell that deletes the Pinecone index. Colab and nbviewer badge URLs now target the main branch.

Reviewed by Cursor Bugbot for commit 2eb5102. Bugbot is set up for automated code reviews on this repo. Configure here.

Port the code from the LlamaIndex integration guide on docs.pinecone.io so the
notebook matches it step by step: pinecone 9.1.0, llama-index 0.14.25, the
llama-index-vector-stores-pinecone package, text-embedding-3-small, and
gpt-4o-mini. Add an out-of-scope evaluation example and a cleanup step.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@nicoalba10
nicoalba10 requested a review from a team as a code owner October 8, 2026 21:51
@cursor

cursor Bot commented Oct 8, 2026

Copy link
Copy Markdown

This PR is too large for Bugbot to review. It changes 1,516 lines and 1,926,351 characters. Split the change into smaller pull requests to get a review.

nicoalba10 and others added 2 commits October 8, 2026 15:56
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@nicoalba10
nicoalba10 merged commit 0faf78b into main Oct 8, 2026
12 checks passed
@nicoalba10
nicoalba10 deleted the update-llamaindex-notebook branch October 8, 2026 22:32
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.

1 participant