yydiao1025 / MacFrag

18 stars 7 forks source link

当smi文件中出现空行时,程序报错 #2

Open lucienshawls opened 1 year ago

lucienshawls commented 1 year ago

您好,我发现了您的代码中的一个bug。 其一,以asMols设置为False时(第468行)为例,当用作输入的smi文件中有空行时,您的程序会出现无法处理的异常。具体来说,据代码第472行推断,您本是想要程序在发现某个mol为无效值 (None) 时跳过(不加入frags中),但是474行的缩进出现了问题,导致如果当前mol为空,那么frags变量未定义,引发474行报错。事实上,只要将474行及循环体向下缩进,即可正常跳过无效mol,已帮您改正。当asMols设置为True时同理。

其二,最开头的几段注释中含有非unicode字符,导致在某些设备上,python无法正常读取这些代码而报错。已在不改变文意的情况下帮您纠正。

具体请见Pull request #1 .