icml18-jtnn icon indicating copy to clipboard operation
icml18-jtnn copied to clipboard

Bug on icml18-jtnn/fast_jtnn/chemutils.py", line 324, in check_aroma

Open huangwb opened this issue 5 years ago • 1 comments

Erro comes: “ if len(benzynes) + len(penzynes) == 0: NameError: name 'benzynes' is not defined ” So I go to Line 324 and check the codes nearby

aenzynes = [get_nid(node) for node in nei_nodes + [ctr_node] if node.smiles in Vocab.benzynes]

It seems that aenzynes should be benzynes

huangwb avatar Apr 08 '19 02:04 huangwb

Hi,

I didn't find the line aenzynes in fast_jtnn/chemutils.py. Could you pull the latest version from github?

Wengong

wengong-jin avatar Apr 10 '19 16:04 wengong-jin