Skip to content

feat: implement localized relationship predicate extraction for graph edges (#799)#803

Open
suhaniiz wants to merge 1 commit into
param20h:devfrom
suhaniiz:feat/relationship-extraction-799
Open

feat: implement localized relationship predicate extraction for graph edges (#799)#803
suhaniiz wants to merge 1 commit into
param20h:devfrom
suhaniiz:feat/relationship-extraction-799

Conversation

@suhaniiz

@suhaniiz suhaniiz commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

📋 PR Checklist

Thank you for contributing to PDF-Assistant-RAG! 🎉
Please fill out this template before submitting. PRs without it filled in will be closed.


🔗 Related Issue

Closes #799


📝 What does this PR do?

Implements a direct local relationship extraction feature (predicate labeling) for the knowledge graph building pipeline.

Previously, the GraphRAG implementation built a standard co-occurrence network, incrementing an edge weight but dropping all semantic contextual information explaining why those entities appeared together. This limited the accuracy of downstream LLM RAG queries.

This PR introduces an extract_relationship_predicate() utility that isolates the textual span between matching entities within a given text chunk. This semantic context description is stored directly within a mutable relationships set on graph edges, which is safely serialized into clean JSON arrays during the graph sanitization phase.


🗂️ Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 🔧 Refactor / code cleanup
  • 📝 Documentation update
  • 🎨 UI / styling change
  • ⚙️ CI / tooling / config change
  • 🧪 Tests

🧪 How was this tested?

  • Tested the affected API endpoints manually
  • Verified that generated edge objects inside the serialized graph data file save with clean "relationships": [...] list string arrays.

📸 Screenshots (if UI change)

N/A (Backend feature)


⚠️ Anything to flag for reviewers?

The predicate window extraction features basic boundary slicing and multi-space scrubbing fallbacks. It runs locally without needing an external LLM, maintaining low ingestion costs and fast building loop benchmarks.


✅ Self-Review Checklist

  • My branch is based on dev, not main
  • I have not added any secrets / API keys
  • I have not modified main branch or any HuggingFace deployment config
  • My code follows the existing style (no unnecessary formatting changes)
  • I have updated relevant docs / comments if needed

@suhaniiz suhaniiz requested a review from param20h as a code owner July 6, 2026 19:16
@vercel

vercel Bot commented Jul 6, 2026

Copy link
Copy Markdown

@suhaniiz is attempting to deploy a commit to the param20h's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

[FEAT] Implement Direct Relationship Extraction (Predicate Labeling) for GraphRAG Edges

1 participant