Skip to content

Add pg_trgm candidates to entity resolution - #67

Open
mikemikimike wants to merge 1 commit into
ayushcodes10:mainfrom
mikemikimike:fix/pg-trgm-entity-resolution
Open

Add pg_trgm candidates to entity resolution#67
mikemikimike wants to merge 1 commit into
ayushcodes10:mainfrom
mikemikimike:fix/pg-trgm-entity-resolution

Conversation

@mikemikimike

Copy link
Copy Markdown

Summary / Problem

Closes #59. Entity resolution currently relies on exact and embedding similarity candidates, which can miss short technical identifiers and abbreviations despite a strong lexical overlap.

Changes

  • Enable PostgreSQL pg_trgm in migration 0022.
  • Add a GIN trigram index for AGE Node canonical names.
  • Add a lexical trigram candidate channel and fuse it with vector candidates using the existing reciprocal-rank fusion helper.
  • Add unit coverage for rank fusion and an integration regression test for identifier abbreviations.

Tests

  • ruff check on changed files: passed (with the repository's container-executable EXE002 limitation excluded).
  • Targeted unit/guard tests: 19 passed.
  • Host git diff --check: passed.
  • Integration regression: skipped because no ECHO_MEMORY_DATABASE_URL / PostgreSQL+AGE service was available.
  • echo-memory calibrate and AUC recalibration were not run for the same unavailable database/service limitation; no AUC value is claimed.
  • Pre-PR gate: prepr-20260913T013430Z-21537, Python 3.11 medium, code-related checks passed; container lacked git.

Compatibility / Known limitations

The migration requires PostgreSQL with Apache AGE and the pg_trgm extension. Existing resolution thresholds were not changed. Full database integration and calibration remain to be run in an environment with the repository's AGE/pgvector database service.

Issue link

Closes #59

@github-actions

github-actions Bot commented Sep 13, 2026

Copy link
Copy Markdown

All contributors have signed the CLA.
Posted by the CLA Assistant Lite bot.

@mikemikimike

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

github-actions Bot added a commit that referenced this pull request Sep 13, 2026
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.

Add a pg_trgm channel to entity resolution

1 participant