Skip to content

Conversation

@tomtung
Copy link
Contributor

@tomtung tomtung commented Oct 7, 2017

No description provided.

@zsdonghao
Copy link
Member

非常感谢您提供的例子,在merge之前,我想我能否把FastTestEmbeddingInputLayer放入 tl.layers 中?若可以的话,我想确认下这个方法是否已经很常用了呢?以及这个方法有没有更好更准确的称呼?

另外,FastTestEmbeddingInputLayer中的Test拼错了?感觉是Text才对?

@tomtung
Copy link
Contributor Author

tomtung commented Oct 9, 2017

不是很确定 fasttext 有多常用,但他们的 repo 确实有10k个星,Keras 包含了 fasttext 的例子(但是实现上有偏差),最近看山杯也有几支获奖队伍用了fasttext或者类似的方法。

如果是放进 tl.layer 里可能是要改成更准确的类名,因为类似的输入层其他模型里也用过,比如DANDCNN论文里类似的网络也作了baseline。

NeuralBagOfWordsInputLayer或者AverageEmbeddingInputLayer

@zsdonghao
Copy link
Member

@tomtung AverageEmbeddingInputLayer 貌似比较好,那我直接merge,把它改了放到tl.layers里吧。谢谢啦~

@zsdonghao zsdonghao merged commit 0eb2015 into tensorlayer:master Oct 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants