Skip to content

docs(graphrag-sdk): expand docs with benchmarks, crash-safety, and examples table#516

Open
gkorland wants to merge 3 commits into
mainfrom
docs/graphrag-sdk-improvements
Open

docs(graphrag-sdk): expand docs with benchmarks, crash-safety, and examples table#516
gkorland wants to merge 3 commits into
mainfrom
docs/graphrag-sdk-improvements

Conversation

@gkorland

@gkorland gkorland commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Improves the GraphRAG-SDK docs page by aligning it with the upstream FalkorDB/GraphRAG-SDK README and docs, filling gaps and clarifying terminology.

Changes

  • Add a Documentation links table (Getting Started, Architecture, Configuration, Strategies, Providers, Benchmark)
  • Add a benchmark-leading accuracy callout linking to docs/benchmark.md (top score on GraphRAG-Bench)
  • Document crash-safety (idempotent rollforward cutover) and concurrency knobs (max_concurrency, update_concurrency) for incremental updates
  • Add a code example demonstrating return_context=True for retrieving the citation/retrieval trail
  • Convert the informal examples list into a table linking all 6 runnable example files
  • Clarify schema vs. ontology terminology — kept schema as the primary term since it matches the actual GraphSchema API, with a note on when 'ontology' applies conceptually

Testing

Documentation-only change; reviewed manually for markdown correctness and verified all links against the upstream repo.

Memory / Performance Impact

N/A

Related Issues

N/A

…amples table

- Add Documentation links table (getting-started, architecture, configuration, strategies, providers, benchmark)
- Add benchmark-leading accuracy callout with link to docs/benchmark.md
- Document crash-safety (rollforward cutover) and concurrency knobs for incremental updates
- Add code example for return_context=True (cited answers)
- Convert examples list into a table linking all example files
- Clarify schema vs. ontology terminology

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Warning

Review limit reached

@gkorland, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 57 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 37f03031-fc73-4e89-bbb1-619ce662791f

📥 Commits

Reviewing files that changed from the base of the PR and between 17d3479 and 1f98a8c.

📒 Files selected for processing (2)
  • .wordlist.txt
  • genai-tools/graphrag-sdk.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/graphrag-sdk-improvements

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@gkorland gkorland requested review from Copilot and galshubeli July 3, 2026 15:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Improves the graphrag-sdk documentation page to better match upstream GraphRAG-SDK docs/README, adding clearer navigation, performance/benchmark context, and more actionable usage guidance.

Changes:

  • Adds a structured “Resources” section with a guide links table plus a benchmark-leading accuracy callout.
  • Expands incremental update documentation with cost model details, crash-safety behavior, and concurrency knobs.
  • Adds a return_context=True retrieval-trail snippet and converts the examples list into a linked table.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
genai-tools/graphrag-sdk.md Expands GraphRAG-SDK docs with guide links, benchmark callout, crash-safety/concurrency notes, retrieval-trail example, and an examples table.
.wordlist.txt Adds “backfill” to the spellcheck allowlist.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

| 3 | [Custom Strategies](https://github.com/FalkorDB/GraphRAG-SDK/blob/main/graphrag_sdk/examples/03_custom_strategies.py) | The benchmark-winning pipeline, ready to drop in |
| 4 | [Custom Provider](https://github.com/FalkorDB/GraphRAG-SDK/blob/main/graphrag_sdk/examples/04_custom_provider.py) | Plug in any LLM or embedder behind a clean interface |
| 5 | [Notebook Demo](https://github.com/FalkorDB/GraphRAG-SDK/blob/main/graphrag_sdk/examples/05_notebook_demo.ipynb) | An interactive walkthrough that shows the provenance trail |
| 7 | [Incremental Updates](https://github.com/FalkorDB/GraphRAG-SDK/blob/main/graphrag_sdk/examples/07_incremental_updates.py) | `update`, `delete_document`, and `apply_changes` for CI-driven graph syncs |
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.

2 participants