Closed
Bug 1200573
Opened 9 years ago
Closed 9 years ago
Add Pocketsphinx speech model to gaia-l10n/es
Categories
(Mozilla Localizations :: es / International Spanish, defect)
Mozilla Localizations
es / International Spanish
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kdavis, Assigned: willyaranda)
References
Details
(Whiteboard: [webspeechapi][vaani][systemsfe])
Attachments
(1 file)
As part of Bug 1178288 the es language pack will include a speech
recognition model for es. Thus, this model should thus be included
in the gaia-l10n/es repository.
I have the models prepared and they simply need to be committed to the repository.
The suggested directory layout is as follows:
es
├── apps
│ ├── bluetooth
│ ├── ...
├── shared
│ ├── branding
│ ├── ...
└── speech-data
├── es.dic
├── es.dic.dmp
├── feat.params
├── mdef
├── means
├── mixture_weights
├── noisedict
├── sendump
├── transition_matrices
└── variances
Addition of Pocketsphinx speech model
Attachment #8655518 -
Flags: review?(willyaranda)
PS: mauricio was unable to be assigned the review as he hasn't signed on in awhile.
Comment 4•9 years ago
|
||
This is great!
I'll keep in touch about this. :)
Willy, if there is something I can help you with, just ask me!
Assignee | ||
Comment 5•9 years ago
|
||
I just requested access to the patch on Drive.
Flags: needinfo?(kdavis)
I just gave access. If you have any further problems with access, let me know.
Flags: needinfo?(kdavis)
Assignee | ||
Updated•9 years ago
|
Attachment #8655518 -
Flags: review?(willyaranda) → review+
Keywords: checkin-needed
Keywords: checkin-needed
Keywords: checkin-needed
Who is responsible for the checkin of this patch?
Flags: needinfo?(willyaranda)
Assignee | ||
Comment 8•9 years ago
|
||
I can land it later today. What is the branch it should land into?
Flags: needinfo?(kdavis)
As far as I know, it should land in the default branch. This is what was
done for the fr speech model[1].
[1] https://hg.mozilla.org/gaia-l10n/fr/rev/d48187deb314
Flags: needinfo?(kdavis)
Assignee | ||
Comment 10•9 years ago
|
||
Ricardo, Inma, are you OK if I land this directly to the gaia-l10n? I'll ping you if I can't :)
Flags: needinfo?(willyaranda)
Flags: needinfo?(rpmdisguise-nave)
Flags: needinfo?(inmitabarrios)
Comment 11•9 years ago
|
||
No problem at my side. I'm currently not dealing with gaia repository.
However, I wonder what will Locamotion do when this patch lands. Specifically, we should make sure that Locamotion is not going to wipe it out when a new commit is done from it, and also that it won't crash when we ask Locamotion to rescan the repository.
We could as well just try and see... }:->
Flags: needinfo?(rpmdisguise-nave)
Reporter | ||
Comment 12•9 years ago
|
||
(In reply to Guillermo López :willyaranda (probably SLOW response) from comment #10)
> Ricardo, Inma, are you OK if I land this directly to the gaia-l10n?
Guillermo did it land on gaia-l10n?
Flags: needinfo?(willyaranda)
Assignee | ||
Comment 13•9 years ago
|
||
Nope, I will do it next week (I'm on PTO without my computer).
Cheers,
Assignee | ||
Comment 14•9 years ago
|
||
https://hg.mozilla.org/gaia-l10n/es/rev/f0dafdfeab79
Done. Anything else from my side?
Status: NEW → ASSIGNED
Flags: needinfo?(willyaranda)
Flags: needinfo?(kdavis)
Flags: needinfo?(inmitabarrios)
Reporter | ||
Comment 15•9 years ago
|
||
(In reply to Guillermo López :willyaranda (probably SLOW response) from comment #14)
> https://hg.mozilla.org/gaia-l10n/es/rev/f0dafdfeab79
>
> Done. Anything else from my side?
No that's it. Thanks!
Flags: needinfo?(kdavis)
Updated•9 years ago
|
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•