No longer works with PET

[goodmami]

There is one line in letypes.tdl (a docstring) that causes flop to segfault. https://github.com/delph-in/jacy/blob/master/letypes.tdl#L1068

Note, I was trying with the 64-bit version of flop in the LOGON distribution, which may not be the same as the 32-bit version. I thought that support for TDL docstrings was added to PET, but I could be wrong.

After fixing the above and flopping the grammar, it still seems broken as cheap is unable to parse a simple sentence:

$ echo "雨 が 降っ た ．" | cheap japanese.grm
[...]
(1) `雨 が 降っ た' [0] --- 1 (0.00|0.00s) <15:24> (502.6K) [0.0s]

There's a lot of warnings when parsing that look like this:

   3189602 WARN: Unknown type/instance `adv-p-lex-naadj' in feature #70942

Note: I'm trying to use PET so I can run a JaEn script (see here https://github.com/delph-in/JaEn/blob/master/utils/extr-rules/extr-rule.bash#L49). I'll probably try and rewrite the script to use ACE, so I think an acceptable solution to this issue is to deprecate and remove PET support, but maybe there are others that still use PET with Jacy?

[fcbond]

I had a quick look, but can't quickly solve it. I suspect it may have something to do with repp/chart mapping, which we have added for ace but maybe not linked for pet. I am not ready to get rid of the pet config files (as it would be nice to have it working) but it is not a high priority.

…

On Fri, Sep 8, 2017 at 6:48 AM, Michael Wayne Goodman < ***@***.***> wrote: There is one line in letypes.tdl (a docstring) that causes flop to segfault. https://github.com/delph-in/jacy/blob/master/letypes.tdl#L1068 Note, I was trying with the 64-bit version of flop in the LOGON distribution, which may not be the same as the 32-bit version. I thought that support for TDL docstrings was added to PET, but I could be wrong. After fixing the above and flopping the grammar, it still seems broken as cheap is unable to parse a simple sentence: $ echo "雨 が 降っ た ．" | cheap japanese.grm [...] (1) `雨 が 降っ た' [0] --- 1 (0.00|0.00s) <15:24> (502.6K) [0.0s] There's a lot of warnings when parsing that look like this: 3189602 WARN: Unknown type/instance `adv-p-lex-naadj' in feature #70942 Note: I'm trying to use PET so I can run a JaEn script (see here https://github.com/delph-in/JaEn/blob/master/utils/extr- rules/extr-rule.bash#L49). I'll probably try and rewrite the script to use ACE, so I think an acceptable solution to this issue is to deprecate and remove PET support, but maybe there are others that still use PET with Jacy? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#47>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABD8xsHNysxDDbJtqxDqtmJOCjEv7L02ks5sgHKtgaJpZM4PQfDo> .

-- Francis Bond <http://www3.ntu.edu.sg/home/fcbond/> Division of Linguistics and Multilingual Studies Nanyang Technological University