Skip to content

Add Japanese data - #209

Open
shama7gupta wants to merge 3 commits into
mainfrom
add-japanese-data
Open

Add Japanese data#209
shama7gupta wants to merge 3 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

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