1065911 - Include default english acoustic and language models on B2G build.

Status: RESOLVED DUPLICATE of bug 1051146

(Firefox OS Graveyard :: GonkIntegration, defect)

Description

[Andre Natal]

English AM and LM should be shipped as default on B2G if some specific flag on mozconfig was set.

Comment 1

[Andre Natal]

Adding Sandip comments from old bug (1051146)

 Sandip Kamat 2014-08-19 18:44:10 PDT

My current thought for existing markets of FxOS is to preload English and Spanish "language packs" by default and add other languages on demand. 2 things to note:

1. OEMs can decide to change this config for their markets/regions
2. They can also preload another (or replace existing) language packs

We should have a table indicating memory footprints for them.

Comment 2

[Andre Natal]

Jonas,

can you please point to me into which source folder that I should include the models? I remember that in our conversation you told me they can't stay at $topsrcdir/models, but in some kind of "resources" folder, for example as dictionaries for localization. 

Can you please help me?

Thanks!

Andre

Comment 3

[Olli Pettay [:smaug] [away July 4-8]]

Wouldn't it make sense to put them close to the speech recognition files.
Similar to what we do for spellchecker's locales 
http://mxr.mozilla.org/mozilla-central/source/extensions/spellcheck/locales/

Comment 4

[Andre Natal]

(In reply to Olli Pettay [:smaug] from comment #3)
> Wouldn't it make sense to put them close to the speech recognition files.
> Similar to what we do for spellchecker's locales 
> http://mxr.mozilla.org/mozilla-central/source/extensions/spellcheck/locales/

Ok, will put them there so.

Comment 5

[Andre Natal]

Attachment: This patch includes the binary acoustic models and patches the build system to put them at /dist/bin/ folder

Comment 6

[Andre Natal]

Comment on attachment 8504467 [details] [diff] [review]
This patch includes the binary acoustic models and patches the build system to put them at /dist/bin/ folder

The patch is too large to upload, since include the acoustic models binary files.

So this is the url to download it:

https://www.dropbox.com/s/sj5oy9totwbkjvh/0002-Include-pocketsphinx-english-models-in-content-media.patch?dl=0

Comment 7

[Olli Pettay [:smaug] [away July 4-8]]

Comment on attachment 8504467 [details] [diff] [review]
This patch includes the binary acoustic models and patches the build system to put them at /dist/bin/ folder

You should upload the build changes as a patch
(and they need a build peer review).


Btw, how much memory these acoustic models will take on b2g?

Comment 8

[Andre Natal]

(In reply to Olli Pettay [:smaug] from comment #7)
> Comment on attachment 8504467 [details] [diff] [review]
> This patch includes the binary acoustic models and patches the build system
> to put them at /dist/bin/ folder
> 
> You should upload the build changes as a patch
> (and they need a build peer review).
> 

Ok, I will upload them split that way we discussed.

> 
> Btw, how much memory these acoustic models will take on b2g?

In this comment we have a brief benchmark in Flame, in exact point where it recognition occurs: https://bugzilla.mozilla.org/show_bug.cgi?id=1051148#c14

Comment 9

[Andre Natal]

Attachment: Patch build system to include acoustic models folder in staging

Comment 10

[Andre Natal]

Attachment: Bug 1065911 - Patch build system to include acoustic models folder in staging

Comment 11

[Andre Natal]

Attachment: Bug 1065911 - Including sendump to english acoustic model folder

Comment 12

[Andre Natal]

Attachment: Bug 1065911 - Including acoustic model binary files

Comment 13

[Andre Natal]

Attachment: Bug 1065911 - Including default cmu07a.dic to models folder

Comment 14

[Olli Pettay [:smaug] [away July 4-8]]

Comment on attachment 8505980 [details] [diff] [review]
Bug 1065911 - Including sendump to english acoustic model folder

The file should go to
dom/media/webspeech/recognition/pocketsphinx/models/en-us-semi
With that, r+

(assuming the rest of the patch are ok.)

Comment 15

[Olli Pettay [:smaug] [away July 4-8]]

Did you ever send the email to dev.platform?

Comment 16

[Olli Pettay [:smaug] [away July 4-8]]

Comment on attachment 8505981 [details] [diff] [review]
Bug 1065911 - Including acoustic model binary files

All these need to go to
dom/media/webspeech/recognition/pocketsphinx/*

Comment 17

[Olli Pettay [:smaug] [away July 4-8]]

Comment on attachment 8505982 [details] [diff] [review]
Bug 1065911 - Including default cmu07a.dic to models folder

Same here

Comment 18

[Olli Pettay [:smaug] [away July 4-8]]

Comment on attachment 8505979 [details] [diff] [review]
Bug 1065911 - Patch build system to include acoustic models folder in staging

And same here. dom/media/webspeech/recognition/pocketsphinx/*


And no patch should land before some dev.platform discussions.

Comment 19

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Olli, can you help here. See comment 2.

Comment 20

[Dietrich Ayala (:dietrich)]

Olli, also can you describe what should be covered in dev.platform?