phetsims / natural-selection

"Natural Selection" is an educational simulation in HTML5, by PhET Interactive Simulations
GNU General Public License v3.0
3 stars 7 forks source link

French (fr) and Kinyarwanda (rw) translations lead to confusing data #357

Closed Nancy-Salpepi closed 2 months ago

Nancy-Salpepi commented 1 year ago

Test device MacBook Air M1 chip

Operating System 13.5

Browser Safari 16.6

Problem description For https://github.com/phetsims/qa/issues/968, the French translations for lessThanValuePercent and valuePercent have a ">" symbol in front of them, instead of a "<" symbol and no symbol, respectively. This leads to data that doesn't make sense.

Visuals In sim:

Screenshot 2023-08-07 at 2 08 51 PM

Here is what the French and Spanish translations look like for comparison:

Screenshot 2023-08-07 at 2 23 42 PM
Nancy-Salpepi commented 1 year ago

I also see the same thing with the locale Kinyarwanda (rw).

jbphet commented 1 year ago

I'm wondering how to address this. I can fix and republish the translations pretty quickly and easily, but then other translators might just make the same mistake. Should we consider fixing up the string somehow so that this doesn't happen? But we have other similar strings and they don't seem to get messed up like this (that I know of). Why this one? Just a fluke?

I'll probably just fix these and republish, then ping @oliver-phet to let the translators know that we had to manually fix their translations.

@oliver-phet - Do you have any thoughts on how to perhaps better handle this?

Also mentioning @pixelzoom, since he's the responsible dev for Natural Selection and might have a thought or two on the matter.

oliver-phet commented 1 year ago

@oliver-phet - Do you have any thoughts on how to perhaps better handle this?

I personally don't think it's worth our effort to fix these unless the translator is unresponsive/no longer active. We should alert the credited translator about the issue and have them manually correct.

pixelzoom commented 1 year ago

I'm just about ready to publish 1.5.0-rc.2, so we need to decide what to do here ASAP. My feeling is that we should decouple this from the 1.5 release, and address it separately. If there hasn't been a decision by Monday 8/14, then I'll consider it decoupled.

I don't care whether a PhET developer fixes the strings, or whether we ask the credited translators to fix. But the latter would be an opportunity for education, so that this doesn't happen again.

jbphet commented 1 year ago

No need to delay the release, as it doesn't impact source code for the sim.

@oliver-phet - Please notify the translators and let them fix it - I think it's a good approach so that they learn from it. Please let me know if you need to know the specific translators that made the commits.

oliver-phet commented 1 year ago

Translator notified.

pixelzoom commented 1 year ago

@oliver-phet You said "Translator notified", singular. Note that there are 2 locales that have been identified as having this problem: French (fr) and Kinyarwanda (rw). So you'll presumably need to notify 2 translators.

I looked through the babel files, and those appear to be the only locales with this problem.

oliver-phet commented 1 year ago

@oliver-phet You said "Translator notified", singular. Note that there are 2 locales that have been identified as having this problem: French (fr) and Kinyarwanda (rw). So you'll presumably need to notify 2 translators.

I looked through the babel files, and those appear to be the only locales with this problem.

Whoops, I missed the comment that this also occurred in the Kinyarwanda translation (I'm going to update the issue title)! I've just emailed the Kinyarwanda translator about the mistake.

pixelzoom commented 3 months ago

@oliver-phet Do you have an update on this? This issue has been open for a long time, would like to get it closed.

oliver-phet commented 3 months ago

No updates. I pinged both the French and Kinyarwanda translators on Feb 14, 2024 reminding them that the correction still hasn't been made. I'll ping again now.

pixelzoom commented 2 months ago

@oliver-phet Any reply from the translators? If not, I'm going to close this issue.

oliver-phet commented 2 months ago

Thanks for the ping! The French translator corrected in https://github.com/phetsims/babel/commit/e6c645d2ae8692b8988cfae13f6baad3db882f10

No word from the Kinyarwanda translator. I think this issue is fine to close.