Port 'LLM example retrieval' #641

Activity

1. Enkidu93

   added

   portingThis issue consists of porting code from machine.py to machine.This issue consists of porting code from machine.py to machine.

   on Nov 12, 2026
2. github-project-automation

   added this to Servalon Nov 12, 2026
3. github-project-automation

   moved this to 🆕 New in Servalon Nov 12, 2026
4. claude commented on Nov 12, 2026

   

   claudeBot

   on Nov 12, 2026 – with Claude

   Last edited by claude

   Contributor

   More actions

   Claude finished @Enkidu93's task in 3m 54s —— View job • claude/issue-641-20261112-1529 • [Create PR ➔](https://github.com/sillsdev/machine/compare/master...claude/issue-641-20261112-1529?quick_pull=1&title=Issue%20%23641%3A%20Changes%20from%20Claude&body=This%20PR%20addresses%20issue%20%23641%0A%0AGenerated%20with%20%5BClaude%20Code%5D(https%3A%2F%2Fclaude.ai%2Fcode)

   ---

   Ported machine.py PR #412 (LLM example retrieval)

   • Read issue and source PR
   • Ported Example, ExampleRetriever, TopKExampleRetriever, TfidfExampleRetriever, CoverageExampleRetriever, ExampleRetrieverFactory and ExampleRetrievalMethod to SIL.Machine.Translation.Llm, plus an internal TfidfIndex
   • Ported the tests (26 new tests, all passing)
   • ./local_check.sh --agent-strict: local_check: PASSED
   • Pushed and opened the PR

   PR: #642 (closes #641)

   Differences from the Python version

   • There is no scikit-learn or numpy equivalent here, so TF-IDF is a small internal sparse implementation that follows TfidfVectorizer defaults. It adds no new dependency.
   • Ties in ranking are broken by corpus order, because numpy leaves tie order unspecified.
   • Subclasses call Fit() at the end of their own constructor, because derived fields are not initialized yet when the base constructor runs.

   Not ported: pyproject.toml, uv.lock, the llm extra, __init__.py and the sklearn availability check. These are Python packaging only.
   · Branch
5. Enkidu93

   closed this as not plannedon Nov 12, 2026
6. github-project-automation

   moved this from 🆕 New to ✅ Done in Servalon Nov 12, 2026