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

Arabic not in Language dropdown #5266

Open
tim-moody opened this issue Aug 30, 2016 · 28 comments
Open

Arabic not in Language dropdown #5266

tim-moody opened this issue Aug 30, 2016 · 28 comments
Assignees

Comments

@tim-moody
Copy link

Summary
Arabic is no longer in the language selection dropdown

System information

Operating system: fedora 22
Version: 0.16.8
Browser: chrome

was able to install manually with kalite manage

@benridge
Copy link

benridge commented Sep 5, 2016

I had the same issue.
Then I realized the dashboard page is showing the incorrect value for Arabic.
Dashboard shows 99% is translated, but the actual amount is 22%.
Not sure if it has reached the threshold for beta, but the interface definitely has translation issues, showing a mix of latin, cyrillic, chinese, and arabic characters in the learning menu (see attached screenshot) .
So the true bug is that the dashboard page is incorrectly calculating the amount translated.
Manually installing is the appropriate installation approach for a language that isn't yet beta.

arabic language pack example

@benjaoming
Copy link
Contributor

@tim-moody @benridge thanks for reporting! It seems that the Arabic language packs aren't being built.

@aronasorman could you build and upload Arabic without rebuilding everything else?

The latest content pack zip is from 25-Mar-2016, which is way before we fixed a series of critical issues in the content packs.

@aronasorman
Copy link
Collaborator

@benjaoming Yup! Can add it. Although be warned that this bug seems to be happening in Khan Academy's side, due to their Arabic translations having erronous strings such as the ones you see here. So it's still not fixed with the latest content pack.

@aronasorman
Copy link
Collaborator

The best course of action here is to go to their Crowdin project ((www.crowdin.com/project/khanacademy)) and fix the strings there.

@benjaoming
Copy link
Contributor

@aronasorman -- I guess the offical label of this content pack is "beta" still as in "beta": true ? :)

@benjaoming
Copy link
Contributor

If you can build and upload just the ar language pack, it's fine... we can update the all_metadata.json manually...

@benjaoming
Copy link
Contributor

@mrpau-eduard @mrpau-richard @cpauya - are you able to build the Arabic language pack?

@mrpau-richard
Copy link
Contributor

mrpau-richard commented Oct 3, 2016

Hi @benjaoming & @benridge Here's the Arabic language pack ar.zip
I built it manually in my local, I will create a PR soon to build it at the build server.

@mrpau-richard
Copy link
Contributor

@benridge & @benjaoming Arabic language pack contain only ar videos and the strings is not fully translated.

screen shot 2016-10-03 at 9 17 22 am

screen shot 2016-10-03 at 8 59 26 am

@benjaoming
Copy link
Contributor

Nice @mrpau-richard!!

@benridge @tim-moody - would you be able to test the new contentpack? You can install it by running kalite manage retrievecontentpack local ar path/to/ar.zip

@tim-moody
Copy link
Author

Finally got a chance to try the ar.zip link above. This screen shot shows the result.

image

@benjaoming
Copy link
Contributor

@tim-moody that doesn't look super Arabic... looks like a bit of everything :)

@mrpau-richard could you have a look at why it seems that Russian (?) and perhaps Chinese is showing up, too?

@tim-moody
Copy link
Author

This is basically what we had before; see my first screen shot.

is this the source https://ar.khanacademy.org/ ?

@tim-moody
Copy link
Author

can someone explain the process for creating a language pack?

@benjaoming
Copy link
Contributor

@tim-moody unfortunately the project is still internal.

https://github.com/fle-internal/content-pack-maker

@aronasorman anything that keeps us from moving it to a public realm?

@tim-moody
Copy link
Author

I have a server headed for a school in Lebanon for Syrian refugee kids. Sure would be nice to have this fixed.

@cpauya
Copy link
Contributor

cpauya commented Nov 6, 2016

Hi @tim-moody - we're still in the process of testing our fixes for the French, Kiswahili, and Portuguese languages. The French lang pack had a similar issue with yours which shows English along with translated stuff. We hope to squeeze testing the Arabic language this week so we can give you feedback if it's also fixed.

@tim-moody
Copy link
Author

Arabic has more speakers than any of those three according to
http://www.vistawide.com/languages/top_30_languages.htm

@mrpau-richard
Copy link
Contributor

Hi @tim-moody & @benjaoming Now I found the issue, why is the Arabic content pack is not properly translated. It's because the Khan academy translation, in Arabic has 0% translated string. I uploaded a sample .pot file from Khan academy translation is Arabic, but it's translated string is mixed with Chinese string.

Find the 數數 sting in the .pot file.
learn.math.arithmetic-ar.po.zip

Here's the sample translated string logs when I build the Arabic content pack in my local.
screen shot 2016-11-08 at 7 39 32 pm

I think we need to help translate or anyone knows about Arabic language in khanacademy translation to fix this issue.

@tim-moody
Copy link
Author

Why does this problem not appear here?

https://ar.khanacademy.org/

@mrpau-richard
Copy link
Contributor

@tim-moody The Arabic content pack will base its content in the Khan academy Arabic api if some video topics will not found it will use the Khan academy English api. And the content pack maker will translate it using Khan academy crowd in translation.

@benridge
Copy link

benridge commented Nov 8, 2016

As @tim-moody pointed out, the video are correctly titled and translated on https://ar.khanacademy.org.

Screenshot from online academy:
image

So it appears the online version is not using the Arabic content pack.

The Arabic text is available in the online version, so what is needed to copy them into the Arabic content pack?

The online videos are the same as in KA-Lite, so the translations exist. It looks like something isn't setup correctly with the Arabic content pack and that this could be fixed without needing new translations as @mrpau-richard suggested.

In other words, the translations exist, but they are not in the right location. How can those be copied to the right place?

@jamalex
Copy link
Member

jamalex commented Nov 8, 2016

The text on https://ar.khanacademy.org/ is pulled directly from the titles of the playlists and videos on the KA Arabic YouTube channel. The translations used in the KA Lite topic tree come from KA's crowdin page. There's no direct mapping between playlists and topics in the topic tree, so it'd be hard to use those, but in principle we could look up the titles of each video in the YT API and use those as fallback titles in the language packs... Would take some work in the content pack maker.

@benjaoming
Copy link
Contributor

Quick update: @mrpau-eduard and @mrpau-richard are going to have a look at updating the content packs w/ data via Youtube API.

We will also look at the possibility of syncing this data back into upstream CrowdIn.

After that, someone can look at translating remaining data, especially topic nodes (the nodes in the sidebar that don't have video or exercises). This can be done via CrowdIn.

@tim-moody @benridge are you Arabic speakers or do you know someone who would be able to do these additional translations through the CrowdIn interface?

@tim-moody
Copy link
Author

As a temporary measure I have created a standalone module for Internet in a Box using ar.khanacademy.org and some javascript along the lines of KAOS. Of course there are no exercises. As a by-product I have a file with all the youtube ids (which I used to download them with youtube-dl).

Another problem for pulling this back into kalite is that at least some of the youtube ids for the ar.ka.org videos do not match the kalite AR language pack.

If you really get fancy, the whole topic tree is in the index.html file which could be parsed, but a non-Arabic speaker like myself is working in the dark.

@benridge
Copy link

Unfortunately I am not an Arabic speaker. I was working at a refugee camp and hoping to deploy KA Lite there but I have since returned home. I don't have access to Arabic speakers any more.

@tim-moody
Copy link
Author

@benridge I would be interested in speaking to you about your experiences with the refugee camp as I am working on a similar project.

@benjaoming
Copy link
Contributor

Quick update here: We have decided to rebuild a lot of content packs and @mrpau-eduard has made an interface with Youtube for retrieving data such as titles and descriptions.

During next week we hope that an Arabic content pack will land for testing with the 0.17 prerelease, which will also become a final release not before long.

Please note that the user interface for KA Lite is only 23% translated: https://crowdin.com/project/ka-lite/ar

@aronasorman is creating a smaller update for the 0.17 translations, but they will largely be identical to 0.16.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants