thunlp / NRE

Neural Relation Extraction, including CNN, PCNN, CNN+ATT, PCNN+ATT
MIT License
810 stars 309 forks source link

PCNN+ATT has no attentions #3

Closed unknownstar88 closed 8 years ago

unknownstar88 commented 8 years ago

Hi. when I read the codes one by one, I found PCNN+ATT don't have any attention part while CNN+ATT has that.

Concretely, "init.h" in the "CNN+ATT" directory mentions the attention-related matrix called "att_W" and "att_W_Dao" which are used and trained in the trainining step of "train.cpp" but in "PCNN+ATT", I haven't found the parts.

Could you check them??