Added support to parse dates from page body - #514
Conversation
|
Thanks so much for adding this @ivbeg. 💯 👍 We will take a look |
|
@ivbeg I have just tried this without waiting on new release and got this errors
|
|
@ilkerceng hi, I will take a look. Could you provide article url too? |
|
@ilkerceng I've tried a few urls and it's ok
Test cmd line tool here I guess it could be about UTF-16 support, but I need an url or several urls to understand how to fix it. |
|
hi @ivbeg, i'm sorry for the latency, i will try to find which url does create the error and put the example url here. |
|
@ivbeg these are the some urls that gives the error. |
|
@ilkerceng thanks. I've found how it happens and patched. Please try latest code from http://github.com/ivbeg/qddate |
|
hi @ivbeg, i don't know how to apply the patch, is it enough to remove qddate lib and reinstall again? |
|
hi @ilkerceng just pull latest code from "http://github.com/ivbeg/qddate" with |
Disclaimer: I am author of qddate lib http://github.com/ivbeg/qddate , it's designed to improve date parsing, to speed up date parsing greatly.
Right now newspaper doesn't extract dates from page text, this patch should fix it.
I've tested patch against Yandex.Zen articles https://zen.yandex.ru/media/id/5a315e1300b3dd77beb64261/kak-ustroen-nekommercheskii-sektor-v-rossii-chast-2-5a3a1e898139ba9e538c916b
They have no dates in metadata, but they have dates inside span field "21.12.2017"
Another site with dates that now could be extracted - https://www.nato.int/cps/en/natohq/news_151395.htm
And so on.