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

Closed
wants to merge 1 commit into 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
@leokim-l
Copy link
Member Author

Various efforts happened in separate branches.

@leokim-l leokim-l closed this Nov 12, 2024
@leokim-l leokim-l mentioned this pull request Nov 21, 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