search

BinLiang-NLP / InterGCN-ABSA

[COLING 2020] Jointly Learning Aspect-Focused and Inter-Aspect Relations with Graph Convolutional Networks for Aspect Sentiment Analysis
https://www.aclweb.org/anthology/2020.coling-main.13/
53 stars 7 forks source link

generate_graph_for_aspect文件与generate_position_con_graph文件的问题 #10

Open ChaiJZ opened 2 years ago

ChaiJZ commented 2 years ago

您好！读了您的代码，generate_graph_for_aspect生成的.graph_af文件,但代码里面对应的却是论文里的A^inter矩阵，反而generate_position_con_graph生成的是.graph_inter文件，对应的却是论文里的A^F。不知道是否是我理解有误，想问下是否两个文件的内容写反了。 而且，intergcn模型的forward函数中，adj对应的应该是论文里的A^F,但看代码貌似对应的是A^inter。 如果有时间，希望能解答一下，十分感谢您的研究工作！

ChaiJZ commented 2 years ago

generate_position_con_graph中涉及到了当前方面词的矩阵与其他方面词矩阵的相加，即adj_matrix += alpha weight other_m，这是式(5)的内容，即A^F的计算公式。

BinLiang-NLP commented 2 years ago

您好！读了您的代码，generate_graph_for_aspect生成的.graph_af文件,但代码里面对应的却是论文里的A^inter矩阵，反而generate_position_con_graph生成的是.graph_inter文件，对应的却是论文里的A^F。不知道是否是我理解有误，想问下是否两个文件的内容写反了。 而且，intergcn模型的forward函数中，adj对应的应该是论文里的A^F,但看代码貌似对应的是A^inter。 如果有时间，希望能解答一下，十分感谢您的研究工作！

您好！ 非常感谢您提出的问题。 首先，我非常抱歉，通过对本地机器的代码，实验log，以及paper公式的检查，我发现我们这里确实是写错了。 您的理解没有问题，两个文件的内容确实是写反了。 相关问题我已在repo上修复。随后我也会抽时间再次审查和纠正代码。 再次感谢您的问题，并再次表示歉意。 如有其它问题，请随时联系我。