Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set up basic infrastructure for testing in chinese, turkish and dutch #59

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

leokim-l
Copy link
Member

@KyranWissink
@gracsh
@aslgraefe
(Hope I tagged the right people), could you pull this branch, add the tests by commiting to this branch? Whenever all of these are done I can merge it into develop. Feel free to contact me for any questions :)

@leokim-l
Copy link
Member Author

As a reference, I copy pasted from the italian (random choice), since the English one is slightly different (is this OK @pnrobinson ?). Whenever you find Italian in the one file there is, translate that (or rather, what is in the same place in the English version...) into the respective language. E.g.

female46yearsInfantileOnset(), new TestOutcome.Ok("La paziente era una donna di 46 anni. L'inizio della malattia avvenne durante il periodo infantile.")),

there is the italian sentence, go to

female46yearsInfantileOnset(), new TestOutcome.Ok("The proband was a 46-year-old woman. Disease onset occurred when the proband was an infant.")),

and translate into chinese/turkish/dutch. Careful that the translation needs to reflect exactly the structure of what you used in the main source code.

@KyranWissink KyranWissink mentioned this pull request Sep 2, 2024
@aslgraefe
Copy link
Member

@leokim-l I see this now, I will get to this later! Sorry for the delay

@aslgraefe aslgraefe mentioned this pull request Sep 16, 2024
@aslgraefe
Copy link
Member

@leokim-l please check my comment in #68 , I can't find the test files for Turkish

@leokim-l leokim-l mentioned this pull request Oct 8, 2024
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.

4 participants