New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

sentic情感知识融合过程，我发现generate_sentic_dependency_graph.py中的生成增强情感的邻接矩阵的算法与论文中的算法1并不相同 #13

Open

JM0011 opened this issue Aug 2, 2023 · 1 comment

JM0011 commented Aug 2, 2023

您好，作者。我在研读代码的过程中，发现generate_sentic_dependency_graph.py中的生成情感增强邻接矩阵算法与论文中的算法1并不相同。因此我在代码中改正成论文中的算法，但似乎造成了梯度爆炸。以下是我改正后的代码和train图，不知是否是我的理解有偏差，希望得到您的回答！

Owner

BinLiang-NLP commented Sep 22, 2023

您好，作者。我在研读代码的过程中，发现generate_sentic_dependency_graph.py中的生成情感增强邻接矩阵算法与论文中的算法1并不相同。因此我在代码中改正成论文中的算法，但似乎造成了梯度爆炸。以下是我改正后的代码和train图，不知是否是我的理解有偏差，希望得到您的回答！

您好！感谢您的问题。
该版本的代码对情感边计算做了改动，以得到更稳定的性能。因此您可以按现版本的代码来运行。
此外，代码如果跟论文完全一致也是可以的，不会出现梯度爆炸。
根据您提供的代码，我看到是否是因为对word进行了多次+1导致的梯度爆炸？例如下图这几行代码：

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment