Skip to content

feat(tier0): Deterministically break pinnedness ties by QNode ID suffix (prefer earlier IDs)#123

Merged
tokebe merged 1 commit intomainfrom
tier0-pinnedness-ordering
Feb 11, 2026
Merged

feat(tier0): Deterministically break pinnedness ties by QNode ID suffix (prefer earlier IDs)#123
tokebe merged 1 commit intomainfrom
tier0-pinnedness-ordering

Conversation

@everaldorodrigo
Copy link
Copy Markdown
Contributor

Objective

Adjust tie-breaking when multiple nodes share the same pinnedness score.

Ensures more predictable start-node selection when pinnedness scores are equal, aligning tie-breaking with normalized-style node IDs that end in an index-like suffix.

Notes

This only affects cases where pinnedness scores are tied; primary selection by pinnedness is unchanged.

@sentry
Copy link
Copy Markdown

sentry bot commented Feb 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
...c/retriever/data_tiers/tier_0/dgraph/transpiler.py 78.74% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@everaldorodrigo everaldorodrigo marked this pull request as ready for review February 10, 2026 23:33
@tokebe tokebe merged commit de7496e into main Feb 11, 2026
5 checks passed
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