thunlp / NRE

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

Wrong Parameters of PCNN+ONE in models+figs #12

Closed KGIERE closed 6 years ago

KGIERE commented 7 years ago

when I use the parameters in provided “models+figs” file to run the "test.cpp" in PCNN+ONE model, the result is apparent error. image 1

Mrlyk423 commented 6 years ago

Maybe you set the wrong parameter?

Mrlyk423 commented 6 years ago

You can check if the size of hidden state, word embedding is the same as that in my paper.