teaddict / turkce-metin-ozetleme

kelime zinciri algoritmasiyla turkce metin özetleme
https://teaddict.net/ozetle
GNU General Public License v3.0
30 stars 2 forks source link

Keep getting java.lang.IndexOutofBoundsException #1

Closed esokullu closed 6 years ago

esokullu commented 6 years ago

with the simple /api/new?contextOfText={some_short_text}

Using Tomcat 8 on Ubuntu 16.04. Also tried with Tomcat 7 and no luck. JDK: OpenJDK 8 (with memory settings finetuned properly)

Please advise.

esokullu commented 6 years ago

This is the URL btw: http://83.149.67.213:8080/turkcemetinozetleme-0.0.1-SNAPSHOT/api/new

teaddict commented 6 years ago

Hi @esokullu , Can you try like this https://github.com/teaddict/teaddict-net/blob/master/ozetle/turkce-metin-ozetleme.html with this examples https://github.com/teaddict/turkce-metin-ozetleme/tree/master/ornek-metinler . Because they are my test set, if they don't work , I will check it again, I have my own server which is running latest version and there is not any problem. If you still have problems, can you please share your test data here :)

teaddict commented 6 years ago

I tried your service , and it works actually: I use Firefox with this addon https://addons.mozilla.org/en-US/firefox/addon/httprequester/ After opening addon , I gave your service url, then I clicked parameters and Name: contextOfText Value: "Türkiye Yaş Meyve ve Sebze İhracatçı Birliklerinden yapılan yazılı açıklamada, "Rusya Federasyonu'nun Türkiye'den sadece 4 firmaya domates ihracatı için izin vermesi kararını kesinlikle kabul etmiyoruz." ifadesi kullanıldı. Rusya Federasyonu’nun bu kısıtlamasının sektörde tekelleşmeye ve haksız rekabete sebep olacağı savunulan açıklamada şunlar kaydedildi: "Uygulama, yaş meyve sebze sektörü ihracatına ve ihracatçısına büyük zararlar verecektir. Ayrıca, bu uygulama üreticiler üzerinde kartel oluşturacak ve üreticilerimiz, ürünlerini ucuz fiyattan satmak zorunda bırakılacaktır. Tek taraflı olarak belirlenen ve hangi temel ve kritere dayandırıldığı açıklanmayan firma seçimleri, sektöre hiçbir katkı yapmayacak ve gayri resmi uygulama alışkanlıklarının da doğmasına sebebiyet vererek hem ihracatçıların hem de üreticilerin büyük mağduriyet yaşamasına neden olacaktır." Türkiye'nin yaş meyve ve sebze üretiminin dünya standartlarının üzerinde olduğu vurgulanarak, "İnsan ve çevre sağlığı odaklı üretim yapan üreticilerimiz, üretim alanlarında tarım teknolojilerinin bütün imkanlarından yararlanarak Ekonomi Bakanlığı ve Gıda, Tarım ve Hayvancılık Bakanlığı ile iş birliği içinde büyük başarılara imza atmıştır. Birlik olarak sektörde önü alınamayacak zararlara sebebiyet vermemesi açısından Rusya Federasyonu’na ihraç edilecek tüm ürünlerde tüm firmalara kısıtlama olmadan izin verilmesini talep ediyoruz."

then click POST and it returns summary.

summary

esokullu commented 6 years ago

Thank you.

I've just found the problem. When there is no punctuation, not even a single dot, try the following:

The former gives 500 Internal Server Error with java.lang.IndexOutOfBoundsException, the latter works.

esokullu commented 6 years ago

It doesn't make sense to use a summarizer for such a short sentence though, so you may ignore :)

teaddict commented 6 years ago

Yes it is known issue. App expects grammatically correct text :)