yya518 / FinBERT

A Pretrained BERT Model for Financial Communications. https://arxiv.org/abs/2006.08097
Apache License 2.0
569 stars 130 forks source link

remove num2words import #13

Closed mangoeyes closed 4 years ago

mangoeyes commented 4 years ago

num2words is not used in the preprocessing and not required

yya518 commented 4 years ago

we do use num2words in the convert_numbers() method.

mangoeyes commented 4 years ago

Hi,

If num2words is necessary, could you include it in requirements.txt?