fastnlp / fastNLP

fastNLP: A Modularized and Extensible NLP Framework. Currently still in incubation.
https://gitee.com/fastnlp/fastNLP
Apache License 2.0
3.07k stars 448 forks source link

CNN文本分类器的几个小问题 #206

Closed tianylin98 closed 5 years ago

tianylin98 commented 5 years ago

这个模型是Yoon Kim, 2014的经典模型,我观察到docstring和签名中有这样几个问题:

xpqiu commented 5 years ago

这个不是严格的follow,你可以实现一个严格版本。另外你发现不对的地方可以直接改,然后pr过来,在dev0.5的分支上改。

xpqiu commented 5 years ago

还有一些模型在这里:https://github.com/fastnlp/fastNLP/tree/dev0.5.0/reproduction/text_classification/model

tianylin98 commented 5 years ago

这个不是严格的follow,你可以实现一个严格版本。另外你发现不对的地方可以直接改,然后pr过来,在dev0.5的分支上改。

好的,我原本觉得先有一个heads-up会比较好。