Skip to content

Feature/lld ir#449

Open
pekopoke wants to merge 6 commits into
MigoXLab:devfrom
pekopoke:feature/lld-IR
Open

Feature/lld ir#449
pekopoke wants to merge 6 commits into
MigoXLab:devfrom
pekopoke:feature/lld-IR

Conversation

@pekopoke

@pekopoke pekopoke commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

feat(retrieval): add search result quality evaluation

  • add relevance, effectiveness, and authority evaluators
  • add standalone and combined executor-based evaluation scripts
  • support query-level and result-level classified outputs
  • improve LLM response parsing and content issue detection
  • set overall weights to 0.7/0.2/0.1
  • add evaluator tests and usage documentation

pekopoke and others added 3 commits July 14, 2026 11:32
- add relevance, effectiveness, and authority evaluators
- add standalone and combined executor-based evaluation scripts
- support query-level and result-level classified outputs
- improve LLM response parsing and content issue detection
- set overall weights to 0.7/0.2/0.1
- add evaluator tests and usage documentation
- add relevance, effectiveness, and authority evaluators
- add standalone and combined executor-based evaluation scripts
- support query-level and result-level classified outputs
- improve LLM response parsing and content issue detection
- set overall weights to 0.7/0.2/0.1
- add evaluator tests and usage documentation

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request introduces comprehensive search result quality evaluation metrics (relevance, effectiveness, and authority) using Dingo's LocalExecutor, along with documentation, example scripts, and unit tests. The review feedback highlights several robust improvements to prevent runtime errors and parsing failures: updating regex patterns in the relevance grader to correctly handle the reasoning field when it is the last key in a JSON object, explicitly verifying that locations is a list before iterating over it, and replacing a risky and inefficient json.dumps call in the authority grader with a safer iteration-based check for DOI strings.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread dingo/model/llm/llm_search_result_relevance.py
Comment thread dingo/model/llm/llm_search_result_relevance.py
Comment thread dingo/model/llm/llm_search_result_relevance.py Outdated
Comment thread dingo/model/llm/llm_search_result_authority.py Outdated
pekopoke added 3 commits July 14, 2026 14:24
- add relevance, effectiveness, and authority evaluators
- add standalone and combined executor-based evaluation scripts
- support query-level and result-level classified outputs
- improve LLM response parsing and content issue detection
- set overall weights to 0.7/0.2/0.1
- add evaluator tests and usage documentation
# Conflicts:
#	dingo/model/llm/llm_search_result_authority.py
#	examples/retrieval/sdk_eval_authority.py
#	examples/retrieval/sdk_eval_effectiveness.py
#	examples/retrieval/sdk_eval_relevancy.py
#	examples/retrieval/sdk_eval_search_result.py
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