Skip to content

Add Japanese data - #209

Merged
shama7gupta merged 4 commits into
mainfrom
add-japanese-data
Aug 27, 2026
Merged

Add Japanese data#209
shama7gupta merged 4 commits into
mainfrom
add-japanese-data

Conversation

@shama7gupta

Copy link
Copy Markdown
Collaborator

Adds japanese data to medical_hr, itsm, and airline domains.

@tiffanyannedo tiffanyannedo 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.

Adding suggestions to the JA set. Translations have been verified and committed directly to the branch

Comment thread data/itsm_aliases/alpha_garage.json Outdated
Comment thread data/itsm_aliases/downtown.json Outdated
Comment thread data/itsm_aliases/downtown_office.json
Comment thread data/itsm_aliases/east_campus_garage.json Outdated
Comment thread data/itsm_aliases/garage_a.json Outdated
"북측 주차장"
],
"ja": [
"ガレージエー",

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.

It looks like 'garage a' and 'main garage' is used in a number of garage selections. Not sure if this is wanted?

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.

I think this is worth addressing, but separately. The aliases are a little bit overlapping and I think that could be a real problem even in English

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.

That's fine, just wanted to bring it up. The aliases provided for JA are fine if we keep the same 'base'. Can take a look again when we address it separately

Comment thread data/itsm_aliases/north_surface_lot.json Outdated
Comment thread data/itsm_aliases/operations_center.json Outdated
Comment thread data/itsm_aliases/riverside_engineering_center.json Outdated
Comment thread data/airline_dataset.json
"first_name": "美穂",
"last_name": "渡辺",
"companion": {
"first_name": "佑樹",

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.

@shama7gupta In Japan it is not customary for spouses to share a last name or for the wife to take on the husband's last name. It would be more realistic if there is also a last_name field for the companion

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.

not sure this could be easily handled, because it informs the user goal and all that, and the specific language on this record is a woman calling about her husband. This should be minor enough to allow?

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.

That's a good point, that could require some major restructuring of the data. It's not a super realistic scenario but it should be fine for this sort of evaluation

Resolves conflicts from the parallel locale additions in #208 (Dutch)
and #207 (Italian). All three PRs appended a new locale key at the same
position in the same files, so every conflict was a false conflict
resolved by keeping all languages.

The 27 conflicted JSON data files were resolved with a three-way
structural merge, which reported zero real disagreements. Verified that
stripping the "ja" keys from the result reproduces origin/main exactly,
that all 663 Japanese keys survive, and that the nl/it/ko/en/de/hi/es/fr
key sets are byte-identical to main.
@shama7gupta
shama7gupta enabled auto-merge August 27, 2026 00:24
@shama7gupta
shama7gupta added this pull request to the merge queue Aug 27, 2026
Merged via the queue into main with commit 356b546 Aug 27, 2026
2 checks passed
@shama7gupta
shama7gupta deleted the add-japanese-data branch August 27, 2026 00:29
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