Architecture cleanup: fix live bugs, then deepen the LLM, eval-record, novelty, arms and process modules
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 8/100
Direzione di ricerca
This is a multi-wave tracking issue, not a single task, so pick one checklist item and read its linked files first. The Wave 1 bugs in tasks run and eval panel-score are the most self-contained, with regression tests that should fail on main. Wave 4 files (llm.py, utils.py, eval/, report/) are a hold zone, so avoid them until the maintainers say otherwise. Done means the item's regression test fails before the change and passes after it.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Why
An architecture review of dokimi found several concepts implemented in more than one place: the LLM call path, reading answer/eval records, spawning subprocesses, parsing arm specs, novelty coverage. The copies have drifted apart, and some of that drift is live bugs on main. This issue lists the fix series and the order it lands in, so reviewers can see the whole plan and contributors know which files will be busy.
Wave 1: bug fixes
Each PR carries a regression test that fails on main.
-
#8
tasks runwith the default relative--output-dircan score a failed cell as passed. The agent writes its reward relative to the task checkout, the framework reads it relative to its own cwd, finds nothing, and falls back toreturncode == 0. -
#5
eval panel-scorejudges every answer with an empty question and empty context: it readsquestion/content, but the answerer writesquestion_text/snippet. Separately, the panel's parse retries replay the same cached reply. -
#10 The trust gate's
retrieval_hit_ratealways passes, because nothing writesretrieval_hit. Fix: the judge and panel stamp it per evaluation, and older eval files report "not measured". -
#11 The CLI offers
openai-codexas a judge provider.Judgerejects it, but only after the answers have been paid for. -
#24 Follow-up to #8: error the cell when its stale run dir can't be cleared.
Wave 2: current model defaults
- #28 One default set on every surface:
gpt-6.1-solfor answers and generation,claude-sonnet-5-5for the judge. Dropgpt-4o-mini. Require litellm ≥ 1.105.0 for prices. Results will not be comparable with older baselines.
Wave 3: low-conflict refactors
- #27 Write answers, eval and arms through
artifacts.save_artifact(validated, atomic). - #22 Merge
gate/soft_gate.pyandgate/hard_gate.py. - #43 Add
questions.v2with a canonicalquestionfield; v1textfiles are still read. Generated sets currently failquestions.v1. - #26 One arm-spec grammar module covering
+and how names relate to specs. - #44 Novelty: the bundle record owns exclusion classes, coverage and the answer-key verdict (today the gate and the analyzer disagree on the unmeasured share). Follow-up:
RunSettingsand a thin CLI. Coverage basis: #101. settings: #105
Wave 4: central modules
Busy files during this wave: llm.py, utils.py, eval/*, report/*.
- One LLM-call module for provider routing, retry, draws, cache and usage, replacing
utils.get_llm. About 5 PRs. Step 1, the module with old entry points delegating: #78. - #80
eval/records.py: a single reader for answer, eval and arms records. -
eval/compare.py: pairing, summaries and significance. This changes published numbers:report generateandreport evalcurrently disagree on the same file.
Wave 5
- Shared CLI model/provider flags, and an arms run module behind a typed run config instead of an
argparse.Namespacehand-off. - #29 (git and misc sites; harnesses follow) One process runner: kills the whole process tree on timeout, decodes UTF-8, takes an explicit env, keeps partial output. Rebuilt on the current
docker_solver. harnesses: #79 docker solver: #104
Wave 6
- Design issue: a harness = a profile + where the runner runs + a transcript parser.
How these land
- Each PR has one purpose. Fixes, behaviour-preserving refactors, and changes to numbers or defaults are never mixed in one PR.
- At most two open PRs per hot area. The next one is rebased after the previous one merges.
- Wave 4 lands back-to-back. Until it is through, please hold PRs that touch
llm.py,eval/*orreport/*, or mention them here.
- Lingua principale
- Python
- Stelle
- 8
- Fork
- 7
- Merge medio
- 1g 55m
- PR unite (30g)
- 126
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di intel/dokimi
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
I maintainer di solito rispondono entro 1 giorno
-
dependencies
Difficoltà 3/5 1-2 giorni Idoneità per principianti 65/100
intel/dokimi#160 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
I maintainer di solito rispondono entro 1 giorno
-
User loop: eval engine contracts (plan/validate/compare/gate/record), 3 skills, GitHub ActionForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 18/100
intel/dokimi#121 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di intel/dokimi
Issue simili
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 85/100
WLAN-Pi/wlanpi-profiler#341 ·
I maintainer di solito rispondono entro 1 giorno
-
Memory leak: the shared `validation_aborted_error` keeps every traceback it is raised throughForse già presa @Fire162 l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
graphql-python/graphql-core#287 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
roman-corgi/ctc-dip#160 ·
I maintainer di solito rispondono entro 1 giorno
-
type-bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
render_findings.py reads and writes in the host's default encoding, so findings.html breaks on a cp1252 Windows hostForse già presa @Aaqibhafeezkhan l’ha presa oggi. Apertabug good first issue mentored python size: XS up-for-grabs
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
dheerajjha/mcp-migrate#366 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno