natasha / yargy

Rule-based facts extraction for Russian language
MIT License
315 stars 40 forks source link

Lock() в Parser #49

Closed kuk closed 6 years ago

kuk commented 6 years ago

C самой первой версии yargy в парсере создаётся и используется Lock https://github.com/natasha/yargy/blob/master/yargy/parser.py#L187 . @dveselov , это точно нужно? Я просто не очень шарю в этой теме. В своей практике вроде бы эта фича мне не пригождалась. Может убрать? Если надо, пользователь может сам создать Lock() при использовании парсера

dveselov commented 6 years ago

Привет, это делает парсер thread-safe, т.е. делает его работу корректной при использовании из нескольких потоков. Можно убрать лок и добавить предупреждение, как сделано в pymorphy2.

kuk commented 6 years ago

Может просто в доке написать, что типа парсер не thread-safe? ))

dveselov commented 6 years ago

Да, я про это и говорю)

dveselov commented 6 years ago

Убрали в https://github.com/natasha/yargy/commit/1334e6dea94ac21ecfdf8001d446e93a76b87e6d