search

kylebgorman / pynini

Read-only mirror of Pynini
http://pynini.opengrm.org
Apache License 2.0
120 stars 26 forks source link

Incompatibility with OpenFst 1.7.9? #29

Closed lguo2020 closed 4 years ago

lguo2020 commented 4 years ago

Looks like openfst 1.7.9 changed signature of GetFarType in fst/extentions/far/getters.h and I'm getting this compilation error when downloading pynini on top of openfst 1.7.9:

In file included from src/pywrapfst.cc:630:
/home/engshare/third-party2/openfst/1.7.9/platform007/ca4da3d/include/fst/extensions/far/getters.h:17:6: note: declared here
 bool GetFarType(const std::string &str, FarType *far_type);
      ^~~~~~~~~~

Is there a quick fix coming?

kylebgorman commented 4 years ago

Pynini is never compatible with more than one OpenFst release at a time. So, find the Pynini release that is compatible with OpenFst 1.7.9 and upgrade to it.

As the README here states, this is just a mirror that I update sporadically and can be used to submit bugs. You should actually obtain the release from pynini.opengrm.org or Conda (conda install -c conda-forge pynini).

(For reference the compatible version is Pynini 2.1.2, I believe.)