Skip to content

Conversation

@andweber
Copy link

The sphinx-stt uses phonetisaurus. It seems that Phonetisaurus changed its api (option names) from version 0.7.8 to 1.0.

added an option:

phonetisaurus_version

for pocketsphinx which determines the correct option names when calling phonetisaurus.
If nothing is given, Phoentisaurus Version is 1.0 assumed. (which changes the current behaviour)

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 49.742% when pulling 369f1a2 on andweber:phonetisaurus_version into eca07ae on jasperproject:jasper-dev.

@G10DRAS
Copy link

G10DRAS commented May 17, 2016

Where did you get Phonetisaurus 1.0?

@andweber
Copy link
Author

sorry 1.0 might not be correct, I mean the github master branch version:
https://github.com/AdolfVonKleist/Phonetisaurus

@coveralls
Copy link

Coverage Status

Coverage increased (+0.07%) to 49.742% when pulling acb5df4 on andweber:phonetisaurus_version into eca07ae on jasperproject:jasper-dev.

@marlemion
Copy link

This should be implemented. On arch, phonetisaurus is already at the new git version. Hence, the approach as indicated on the website / installation documentation does not work anymore under arch!

For example, even the command phonetisaurus-g2p is not present anymore. There are either phonetisaurus-g2pfst or phonetisaurus-g2prnn. symlinking phonetisaurus-g2p to phonetisaurus-g2pfst does not work, as jasper spits out the line:

ERROR:client.g2p:Command 'phonetisaurus-g2p --model=/xxxx/g014b2b/g014b2b.fst --input=/tmp/tmpAv6vIx.g2p --words --isfile' return with exit status 1

phonetisaurus-g2p --model=/xxxx/g014b2b/g014b2b.fst --input=/tmp/tmpAv6vIx.g2p --words --isfile
GitRevision: kaldi-dirty
FATAL: SetFlags: Bad option: --input=/tmp/tmpAv6vIx.g2p

@G10DRAS
Copy link

G10DRAS commented Feb 16, 2018

Phonetisaurus github version is not compatible with Jasper.
Jasper code need some changes.

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