Skip to content
This repository has been archived by the owner on Sep 7, 2022. It is now read-only.

@jmgilmer Trying to reach out #1

Open
hochshi opened this issue Jun 21, 2018 · 0 comments
Open

@jmgilmer Trying to reach out #1

hochshi opened this issue Jun 21, 2018 · 0 comments

Comments

@hochshi
Copy link

hochshi commented Jun 21, 2018

@jmgilmer Hi,

I'm trying to figure out why this line appears in the code.

mpnn/set2vec.py

Line 156 in 4a1f0dd

attention_w2 = tf.get_variable(name + "att_W_2", [node_dim, node_dim])

It doesn't appear in the MPNN or vinyalis et al. (2016) Order Matters, as far as I can understand. Can you please correct me or provide clarification as to why include this added attention matrix?

Thank you,
Shi

@hochshi hochshi changed the title Trying to reach out @jmgilmer Trying to reach out Jun 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant