thunlp / OpenKE

An Open-Source Package for Knowledge Embedding (KE)
3.83k stars 985 forks source link

Undefined name 'conR' in train_transr_WN18RR.py #186

Closed cclauss closed 5 years ago

cclauss commented 5 years ago

This variable is used before it is defined.

flake8 testing of https://github.com/thunlp/OpenKE on Python 3.7.1

$ flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics

./examples/train_transr_WN18RR.py:31:93: F821 undefined name 'conR'
parameters["transfer_matrix"] = np.array([(np.identity(50).reshape((50*50))) for i in range(conR.get_rel_total())])
                                                                                            ^
1     F821 undefined name 'conR'
1

E901,E999,F821,F822,F823 are the "showstopper" flake8 issues that can halt the runtime with a SyntaxError, NameError, etc. These 5 are different from most other flake8 issues which are merely "style violations" -- useful for readability but they do not effect runtime safety.

THUCSTHanxu13 commented 5 years ago

I have fixed this bug. Please try it again.

cclauss commented 5 years ago

Testing in #189 ...

Samuel123-hub commented 4 years ago

我已修复此错误。请再试一次。

大佬,我在运行Traansr模型时出现错误,segmentation fault(core dump),怎么解决?