Skip to content

data(scenarii): translate 76 RU rows via DatasetUpdater - #336

Merged
jsboige merged 1 commit into
masterfrom
data/scenarii-ru-translation
May 23, 2026
Merged

data(scenarii): translate 76 RU rows via DatasetUpdater#336
jsboige merged 1 commit into
masterfrom
data/scenarii-ru-translation

Conversation

@jsboige

@jsboige jsboige commented May 23, 2026

Copy link
Copy Markdown
Contributor

Summary

Verification

  • Contamination check: 0 hits for barater/baratin/peacher
  • No BOM added to CSV
  • Diff: 74 insertions, 74 deletions (chirurgical — only RU columns changed)
  • Config restored: Mode back to WebBasedImageGeneration|QuestPdfGeneration, RU task disabled

Test plan

  • Cell-by-cell review of RU translations (ai-01 gating)
  • Verify no EN/PT/FR fields modified
  • Confirm 2-4 remaining empty rows are acceptable for P4 quality pass

Part of issue #335 (P1 Scenarii RU/PT).

🤖 Generated with Claude Code

76 previously-empty Russian translation rows filled via DatasetUpdater
with gpt-5.5 model. 163/167 rows now have complete RU fields (title,
smoothTalker, drawer, context, issue, suggestion). 2-4 rows remain
empty for quality pass (P4).

No contamination detected (0 barater/baratin/peacher hits).
No BOM added. Diff: 74 insertions, 74 deletions (chirurgical).

Part of issue #335 (P1 Scenarii RU/PT).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@myia-ai-01

Copy link
Copy Markdown

✅ Coordinator gate (ai-01) — PASS, cell-by-cell verified

Verified against origin/master by parsing both CSV versions (csv module), not by trusting the body. RU columns guarded = indices 20–27 (category_rusuggestion_ru).

Check Result
Non-RU cells changed (FR 1–11, EN 12–19, PT 28–35, edition 36–38) 0 — EN (#322+#330), PT, FR golden master fully preserved
Contamination barater/baratin/peacher in RU cells 0
BOM none (file starts pat…, no EF BB BF)
Diff surgicality 74+/74−, 1 file, only RU columns touched
RU regressions (filled→emptied) 0
Glossary Болтун used 151× in prose (context_ru 103 / issue_ru 48 / suggestion_ru 1); correctly absent from smoothTalker_ru (that column holds character names, e.g. «Мать Цезаря»)
Coverage 74 rows / 434 RU cells filled; 163/167 complete
Non-Cyrillic titles only 2+1 (3.3.5) and 5G (5.3.4) — legitimate language-neutral tokens
CI build Debug + Release + GitGuardian all green

Minor — P4 quality pass only (NOT blocking this merge, tracked in #335)

  • 4 partial RU rows (prose fields still empty): 1.2.6 The Queen's kale, 3.3.2 Rock-solid marriage, 3.3.3 The fake inheritance, 7.2.2 The Poisoned Garden.
  • 3 cells use оратор/софист for the role instead of Болтун — may be legitimate context; verify during P4.
  • Pre-existing (EN, untouched here): 5.3.4 EN title is 5g while FR is «La Conspiration de la 5G» — separate EN data-quality nit, out of this PR's scope.

Verdict: ready to merge. Awaiting explicit GO from @jsboige (per standing rule: ai-01 does not push master without GO). After merge → po-2023 launches PT (same procedure, same gate).

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