Skip to content

Pr/multilingual#121

Open
raghavm243512 wants to merge 21 commits into
mainfrom
pr/multilingual
Open

Pr/multilingual#121
raghavm243512 wants to merge 21 commits into
mainfrom
pr/multilingual

Conversation

@raghavm243512
Copy link
Copy Markdown
Collaborator

@raghavm243512 raghavm243512 commented May 18, 2026

initial multilingual version

Easily extendable to many language using the add_culture_data script. This will do translation, gender consistent naming, suggest names, extend data, etc. So if anyone wants to run a language not committed in EVA data, it is trivially easy to do so.
Readme section showing basic of adding a language.

This adds:
Multilingual data schema and content (initial utterances, system prompt, name aliases)
multilingual support in code
Prompt updates to support multi languages
Script to "add a language" with high degree of automation
WER metric normalization rules, dynamically set per language and creatable via LLM through adding script
Automatic .env.example adjustments (maintains config app accuracy)

Still TODO:

  • Currencies
  • Phone numbers (airline only problem)
  • Actually committing the translations (didn't want to burn credits until finalized)
  • Analysis
  • Testing a large variety of models to ensure they actually get the language code they expected (es-MX vs es, for example)

Copy link
Copy Markdown
Collaborator

@katstankiewicz katstankiewicz left a comment

Choose a reason for hiding this comment

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

can you also add ensure_ascii=False to AuditLog save()

Comment thread src/eva/utils/culture.py Outdated
Comment thread scripts/run_text_only.py Outdated
Comment thread scripts/run_text_only.py Outdated
Comment thread src/eva/assistant/pipeline/services.py Outdated
Comment thread scripts/add_culture_data.py Outdated
Comment thread scripts/add_culture_data.py
Comment thread scripts/add_culture_data.py Outdated
Comment thread scripts/add_culture_data.py Outdated
@raghavm243512 raghavm243512 marked this pull request as ready for review May 28, 2026 21:14
de: Hallo! Wie kann ich dir heute helfen?
en: Hello! How can I help you today?
es: ¡Hola! ¿En qué puedo ayudarte hoy?
fr: Bonjour ! Comment puis-je vous aider aujourd'hui ?
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.

We don't have fq?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

easy to add, will do. The existing languages are just things I saved from testing because why waste them

Comment thread configs/prompts/judge.yaml
Comment thread configs/prompts/judge.yaml Outdated
Comment thread configs/prompts/judge.yaml
Comment thread configs/prompts/simulation.yaml Outdated
Comment thread configs/prompts/simulation.yaml Outdated
Comment thread configs/prompts/simulation.yaml Outdated
Comment thread configs/prompts/simulation.yaml Outdated
Comment thread configs/prompts/simulation.yaml Outdated
Comment thread configs/prompts/simulation.yaml Outdated
Comment thread data/itsm_scenarios/1.json Outdated
Comment on lines +408 to +409
"downtown",
"engineering center",
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.

Since we have this alias in English, I think we should also add the translations for these two in other languages.

Also, can you sort translation by language please?

Copy link
Copy Markdown
Collaborator Author

@raghavm243512 raghavm243512 Jun 1, 2026

Choose a reason for hiding this comment

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

What do you mean by the first part? there are translations in other languages?

"Main Garage"
"a garage",
"main garage",
"garage a",
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.

Wouldn't that match with the name already?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Not sure what you mean

"name_aliases": [
"Downtown",
"Engineering Center"
"downtown",
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.

Also, since we have all these alias in each file, shouldn't we put all the aliases in a separate common file?

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.

3 participants