search

IBM / Graph2Seq

Graph2Seq is a simple code for building a graph-encoder and sequence-decoder for NLP and other AI/ML/DL tasks.
Apache License 2.0
237 stars 61 forks source link

Some issues about aggregators.py file #4

Open ayanesakura opened 4 years ago

ayanesakura commented 4 years ago

In the GatedMeanAggregator class ,I find that when parameter "concat" was False, I can't create it.

Because the attribute “output_dim” of the class will be set only when "concat" is True,while when create self.vars['bias'] variable, self.output_dim is needed,but the class still doens't have this attribute.

Souce code is as follows: ... if concat: self.output_dim = 2 * output_dim ... if self.bias: self.vars['bias'] = zeros([self.output_dim], name='bias') ...

I'm not so good at English ,sorry for my poor expression.

Tanjie1102 commented 4 years ago

可以一起交流吗，加QQ1085347506 或者是QQ邮箱也可以

Tanjie1102 commented 4 years ago

请问你知道数据中的 word2idx那个文件的意义吗？

ayanesakura commented 4 years ago

不好意思，一直没上Gmail,QQ加你了。word2idx是保存字典的文件。

Tanjie1102 notifications@github.com 于2019年10月27日周日 下午10:43写道：

请问你知道数据中的 word2idx那个文件的意义吗？

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/IBM/Graph2Seq/issues/4?email_source=notifications&email_token=AJV7ATFXURMWUAZYQFHVHLDQQWSHVA5CNFSM4JAYUS42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECK756Q#issuecomment-546701050, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJV7ATGUFQDN7LT657DPUJ3QQWSHVANCNFSM4JAYUS4Q .

teddylfwu commented 3 years ago

Dear All,

Please see our newly released graph4nlp library: https://github.com/graph4ai/graph4nlp, which have implemented many GNN methods like Graph2Seq and Graph2Tree models.