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

How to pad the graph if the size of each graph is different #7

Open Barcavin opened 4 years ago

Barcavin commented 4 years ago

Hi,

I am trying to implement the SQL-to-Text paper with the Graph2Seq model on WikiSQL dataset. When preparing the g_ids for the graph, it seems that we need to pad the nodes from all the graphs to the same size. I am wondering if I just need to create PAD nodes isolated or the PAD nodes have to be connected to the graph in some way.

Thanks

694040837 commented 3 years ago

Hi, I have the same question now. Do you solve this problem now?

Barcavin commented 3 years ago

not really. tried several ideas but none can reproduce the paper

On Tue, Jan 12, 2021 at 3:15 AM Listener notifications@github.com wrote:

Hi, I have the same question now. Do you solve this problem now?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/IBM/Graph2Seq/issues/7#issuecomment-758486817, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGNQQITC2XZPTYA3TPPDNPDSZQALLANCNFSM4OFEL2RQ .

694040837 commented 3 years ago

all right..Does the way 'create PAD nodes isolated' work? I tried this way, but my acc is 0....maybe some bugs happen

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.

LN-Curiosity commented 1 year ago

all right..Does the way 'create PAD nodes isolated' work? I tried this way, but my acc is 0....maybe some bugs happen

hi, do you have other solutions to deal with this "PAD nodes" problem?

LN-Curiosity commented 1 year ago

not really. tried several ideas but none can reproduce the paper … On Tue, Jan 12, 2021 at 3:15 AM Listener @.***> wrote: Hi, I have the same question now. Do you solve this problem now? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#7 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGNQQITC2XZPTYA3TPPDNPDSZQALLANCNFSM4OFEL2RQ .

hi, do you have a new idea to work out this problem?