Uh oh!
There was an error while loading. Please reload this page.
Fix stale Parquet references and drop intro flow diagram - #618
Merged
Conversation
- Update intro step list from Parquet to JSONL to match the schema-based/Documents API flow already used elsewhere in the notebook - Remove the outdated "data flow" image and its lead-in sentence
jennapederson
marked this pull request as ready for review
September 3, 2026 22:12
The old /guides/data/understanding-imports URL is superseded by /guides/index-data/import-data.
HEAD requests don't follow redirects by default, and this path is an intentional 308 permalink to the current API version so links don't need updating on every doc release. Scoped narrowly to this path only — not a general allowance for redirecting links.
The notebook uses pd.DataFrame but the install cell never listed pandas, so a clean environment (like the run-notebook CI action) fails with ModuleNotFoundError. Pinned to 3.0.5 to match docs/pinecone-reranker.ipynb.
jennapederson
commented
Sep 3, 2026
ContributorAuthor
These failures are expected at this point as the failing link is valid and there is nothing setup to set against a storage integration with S3. |
Uh oh!
There was an error while loading. Please reload this page.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Note
Low Risk
Documentation and CI allowlist changes only; no runtime product or auth logic.
Overview
Updates
docs/pinecone-import.ipynbso the intro matches the notebook’s actual Documents API / JSONL bulk-import path: solution steps 3–5 now say JSONL instead of Parquet, the imports guide link points toguides/index-data/import-data, and the outdated data-flow diagram (embedded image + lead-in) is removed.pandasis added to the install cell so dependencies match the code that already uses a DataFrame.In
.github/actions/check-links/check-links.py,docs.pinecone.io/reference/api/latest/is added toignore_linksso CI does not fail on Pinecone’s intentional “latest” API permalink redirects (scoped to that path only).Reviewed by Cursor Bugbot for commit 4b4578b. Bugbot is set up for automated code reviews on this repo. Configure here.