-
Notifications
You must be signed in to change notification settings - Fork 122
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
Alignment Failed error with Given Example #36
Comments
What platform was this, and what version of OpenFst? Could you also post the first few lines of the input lexicon. Also the python script you ran is just a wrapper around the c++ binaries. What happens if you just run the aligner:
I just downloaded everything and recompiled it from scratch on my MacBook (OSX 10.12.6, OpenFst 1.6.2) and it seemed to go OK. |
The OpenFst version is 1.6.3 and the OS is RHEL Server release 6.9 (Santiago). The command you tried does work for me too. Took 12 iterations or so. What fails is phonetisaurus-train (with seq2_del or seq1_del) The first few lines of my cmudict.formatted.dict are:
|
@AdolfVonKleist you sure meant to close this? |
@prashantserai I had the same problem when installing OpenFST 1.6.8. After I uninstalled everything and reinstalled using OpenFST 1.6.2 and it worked fine. |
@prashantserai sorry I did not get the notification with your setup details. @wael34218 were you also running RHEL 6.9? The only OpenFst and OS combinations currently running on TravisCI are those described in the config file: I'll try to find some time this week to upgrade OpenFst to 1.6.8, but that will still only cover the existing OSX and Ubuntu 14.04 platform builds. @prashantserai if you can contribute a RHEL configuration addon for the TravisCI yaml that would certainly be welcomed. |
@AdolfVonKleist I am using Ubuntu 16.04 |
FYI I did try changing from OpenFST 1.6.3 to 1.6.2 too, but the problem from the original post persisted for me. I did a verbose log: [serai@zirconium example]$ phonetisaurus-train --lexicon cmudict.formatted.dict --seq2_del --verbose |
I was able to make the recipe in README.md work finally with the following changes after installation (needed a couple of separate hacks for installation i.e. config and build): phonetisaurus-train, commenting out lines 191-194 as below to conquer runtime error
phonetisaurus-apply, line 320 change to conquer syntax error
Modified to:
Don't know why these issues are exclusively on my system cos this is Python code which I thought should've been largely Platform independent. Anyway, hope this helps. PS: The hacks used to build were:
|
Interesting. Thanks for the update. What python version/environment are you running on, also can you confirm the OS/build? I'm surprised that comprehension does not work. I would definitely like to sort out all the python issues; as you say that code should really be platform independent, but I currently do not know how to replicate these issues on my side. |
Hi! this is the info below, I guess it's an old python version [serai@zirconium ~]$ python |
To offer another data point, I also had the same error. For me simply commenting out the 4 lines in the phonetisaurus-train "makeAlignerCommand" function, per @prashantserai above, was all I had to do to move forward. |
Hi!
After some efforts (tweaking Makefiles for Phonetisaurus and MITLM, amongst others), I managed to install Phonetisaurus, but it gives me the following error when trying to run the example from the README.
$ phonetisaurus-train --lexicon cmudict.formatted.dict --seq2_del
INFO:phonetisaurus-train:2018-05-30 17:28:00: Checking command configuration...
INFO:phonetisaurus-train:2018-05-30 17:28:00: Checking lexicon for reserved characters: '}', '|', '_'...
INFO:phonetisaurus-train:2018-05-30 17:28:00: Aligning lexicon...
ERROR:phonetisaurus-train:2018-05-30 17:28:00: Alignment failed. Exiting.
Any ideas what I could do?
The text was updated successfully, but these errors were encountered: