natasha / yargy

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

Added constant attribute support to fact object #46

Closed dveselov closed 6 years ago

dveselov commented 6 years ago

Closes #11

kuk commented 6 years ago

Сейчас можно записать

dictionary({'тысяча'}).interpretation(
    Money.base.normalized(10**3)
)

Идея была в том, что можно писать

normalized()
normalized('value')
normalized(function)

Здесь предлагается для normalized('value') сделать алиас constant? А чем не нравится normalized('value')?

dveselov commented 6 years ago

О, на normalized я не обратил внимания. Собственно, это как раз то, что нужно. Но немного пугает такая неявность (мне почему-то казалось, что normalized - это как inflected, но без склонения). Может быть, разделим normalized на подтипы?

kuk commented 6 years ago

normalized без аргументов это как inflected без склонения. Названия как в прошлой версии yargy.

Потом понадобилось добавить кастомную нормализацию. Я подумал что прикольно оформить это через уже существующий normalized.

Вообще может правда лучше заменить

normalized('value') -> const('value')
normalized(function) -> custom(function)