KNP seems to ignore the `timeout` option about pyknp

[nobu-g]

Thanks!
timeout attribute of KNP class is not used and this is apparently a bug.
I will fix it.

[CLRafaelR]

@nobu-g

Done, I submitted the issue: ku-nlp/knp#7

[nobu-g]

I could not reproduce your issue...
pip supports pyproject.toml and does not require setup.py or __version__.py.
https://github.com/pypa/pip/blob/main/NEWS.rst#features-18

Could you upgrade pip and try again?
Or in another way, you can install the latest pyknp just by pip install pyknp.

[CLRafaelR]

Sorry for my late reply. I successfully installed the latest pyknp via pip without any copy-and-paste procedure after upgrading pip. Thank you for your help!

[CLRafaelR]

@nobu-g

I appreciate your attendance!

[CLRafaelR]

@nobu-g

I don’t think that -timeout option properly works on bash either. The example sentence I mentioned above is parsed, although I set 0 sec for the timeout. Would you mind rechecking whether timeout option of knp itself functions, too?

echo "このような句読点が少ない文章は文章の流れをつかみにくく読みづらいという欠点を持つが書く人はこの方が早く書けることもあるので「句読点が多い文章」よりは出現しやすくさらにパソコンやワープロなどで書き印刷する場合は紙のスペースの節約になり省資源にも多少は貢献すると思われるのみならず最近の女子高生が携帯電話からブログに書いた文章においては句読点がないのが普通だったりするんだけれども読みやすさ第一を心がける場合においては句読点をつけすぎなさすぎないことはやめましょう。" | jumanpp | knp -tab -anaphora -semantic-head -timeout 0

[nobu-g]

Could you make an issue on ku-nlp/knp?

[CLRafaelR]

Thank you for the bug fix.

I want to reinstall pyknp via the following command, but this command does not work since the GitHub repo is missing setup.py and __version__.py

pip install git+https://github.com/ku-nlp/pyknp.git@master#egg=pyknp

So currently, I got the latest stable release of pyknp (0.4.6) from here, copied setup.py and __version__.py of the stable version and placed these two files to my local git repo of pyknp. I succeeded in the reinstallation and I'm now trying to use the timeout option.

[CLRafaelR]

@nobu-g

I confirmed that the bug is fixed in my local environment too. I appreciate your help and maintenance!