Skip to content

data(scenarii): fill 7 partial EN translation gaps - #291

Merged
jsboige merged 1 commit into
masterfrom
data/scenarii-en-partial-gaps
May 17, 2026
Merged

data(scenarii): fill 7 partial EN translation gaps#291
jsboige merged 1 commit into
masterfrom
data/scenarii-en-partial-gaps

Conversation

@jsboige

@jsboige jsboige commented May 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fix 7 records with partial English translations in Scenarii CSV
  • All 7 records had some EN fields filled but were missing 1-3 fields each

Details

Record Fields Fixed
Les droits de l'homme drawer
Shéhérazade drawer
Emménager ensemble context, issue, suggestion_en
Bébé context, issue, suggestion_en
Mariage ? Non merci. context, issue, suggestion_en
La potion de trop title
Cascade primaire drawer

Scenarii EN Status After Fix

  • 91 records fully complete (up from 84)
  • 0 partial records
  • 76 records still empty (require DatasetUpdater for full translation)

Test plan

  • CSV parsing validated — 167 rows, 38 columns, comma delimiter
  • Only 13 field values changed across 7 rows — no data corruption
  • Pipeline regeneration deferred to Phase 1 full translation run

🤖 Generated with Claude Code

Fix partial English translations in Scenarii CSV:
- pk "Les droits de l'homme": add drawer
- pk "Shéhérazade": add drawer
- pk "Emménager ensemble": add context, issue, suggestion_en
- pk "Bébé": add context, issue, suggestion_en
- pk "Mariage ? Non merci.": add context, issue, suggestion_en
- pk "La potion de trop": add title
- pk "Cascade primaire": add drawer

EN now has 91 complete records, 0 partial, 76 empty (full translation
needed via DatasetUpdater). No data corruption — only 13 field values
changed across 7 rows.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

@clusterManager-Myia clusterManager-Myia left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fills 7 partial EN translation gaps in Scenarii CSV. Description accurately lists the 7 records and fields fixed. PR claims only 13 field values changed across 7 rows — consistent with targeted translation completion.

BOM note: same as #292/#293.

@jsboige
jsboige merged commit 2b0de3a into master May 17, 2026
3 checks passed
@jsboige
jsboige deleted the data/scenarii-en-partial-gaps branch May 17, 2026 14:32
jsboige added a commit that referenced this pull request May 17, 2026
Rebased onto master (post PR #291 merge). Standardize category and
subcategory translations across 167 Scenarii records for all 3 languages.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
jsboige added a commit that referenced this pull request May 17, 2026
- Scenarii row 9 (path 1.2.4): "Olympus of Gouge" → "A suffragette" (EN drawer)
- Scenarii row 9 (path 1.2.4): "Um sufrágio" → "Uma sufragista" (PT drawer_pt)
- Strip UTF-8 BOM from Virtues CSV (introduced by PR #290)
- Strip UTF-8 BOM from Scenarii CSV (introduced by PR #291)

Olympe de Gouges is a French historical figure (suffragette), not a mountain.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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