kylebgorman / pynini

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

Composition fails #24

Closed lxkain closed 4 years ago

lxkain commented 4 years ago

When composing two FSTs I get

TypeError: unsupported operand type(s) for *: 'pynini.Fst' and 'pynini.Fst'

kylebgorman commented 4 years ago

tl;dr use '@' instead.

Infix binary operator '*' was always a Python 2 hack, with '@', added early in Python 3, as the ideal representation of composition, since it both looks like LaTeX $\circ$, the standard representation of the operator in textbooks, and is also what was used by Thrax. But Python 2.7 does not support this operator, and this is the first version of Pynini to begin to drop Python 2 support.

K

On Fri, Feb 28, 2020 at 3:00 PM Alexander Kain notifications@github.com wrote:

When composing two FSTs I get

TypeError: unsupported operand type(s) for *: 'pynini.Fst' and 'pynini.Fst'

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/kylebgorman/pynini/issues/24?email_source=notifications&email_token=AABG4OKBV5IXKT3NCETUFK3RFFUOPA5CNFSM4K522IR2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IRG42KQ, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABG4OLVYJ6POIHQ2YCJXMTRFFUOPANCNFSM4K522IRQ .