search

AdolfVonKleist / Phonetisaurus

Phonetisaurus G2P
BSD 3-Clause "New" or "Revised" License
454 stars 122 forks source link

ReadMe has incorrect configure step for openfst. #2

Closed taji closed 7 years ago

taji commented 9 years ago

This configure command 

./configure --enable-compact-fsts --enable-const-fsts \
    --enable-far --enable-lookahead-fsts --enable-pdt --ngram-fsts

Should be:

./configure --enable-compact-fsts --enable-const-fsts \
    --enable-far --enable-lookahead-fsts --enable-pdt --enable-ngram-fsts
taji commented 9 years ago

And just to be thorough on the entire build and test process as described in the ReadMe: (installing under Ubuntu):

  1. Make sure you have the following installed via apt-get: build-essential, python, python-dev, python-setuptools, python-twisted.
  2. Make sure your LD_LIBRARY_PATH environment variable is set to : /usr/local/lib.
taji commented 9 years ago

Finally one more thing, this command:

phonetisaurus-arpa2wfst-omega --lm=test.arpa --ofile=test.fst

returns a 1 exit code (rather than a 0). Should it?

I'm building a Docker container and adding this line as a RUN command in my Dockerfile causes the Docker build to break. Right now I'm working around the issue by invoking the command like this:

phonetisaurus-arpa2wfst-omega --lm=test.arpa --ofile=test.fst || true

BTW, love the twisted service implementation. Makes rpc a snap :-)

theja0473 commented 8 years ago

It is not working for my RPI. errorcapture

theja0473 commented 8 years ago

could you please advice another ./configure command

theja0473 commented 8 years ago

I am using bellow g++ version. g version

AlJohri commented 8 years ago

@theja0473 what are you doing?? why are you commenting on 3 different unrelated issues?

dmbuchta commented 8 years ago

I'm still having a configuration error with openfst using the updated ReadMe. Please see #10, any advice would be appreciated.

AdolfVonKleist commented 7 years ago

This should now be resolved thanks to @AlJohri pull request.