GitLab

@zdcao 一些修改附加在每个点后面的commit hash 了，感谢！

• 1）首先你得显示地写一个main.py文件，不然要理清你的代码逻辑还得一个一个去翻各个文件做了什么；c8089fea

• 2）在import的时候把你写的和外部库分开，https://code.itp.ac.cn/Osgood/openlam-hack/-/blob/main/src/relax.py#L10 比如这里就混在一起了 c8089fea

• 3）考虑默认参数的合理性。https://code.itp.ac.cn/Osgood/openlam-hack/-/blob/main/src/relax.py#L160-161 这里fmax取的是0.001，但是steps是200 (作为参考matgl tutorial给的默认参数是fmax=0.01和steps=500)，你可以输出fmax和Energy的曲线，看看fmax从0.01到0.001的时候能量是否会发生较大改变，步数又是否合理。最后算Ehull关心的其实是能量 c8458451

• 4） 比对formula这块是否写过测试，这里formula的类型是str，没有办法处理元素顺序不一样的情况。https://code.itp.ac.cn/Osgood/openlam-hack/-/blob/main/src/filter.py#L32-33 f00aefa5

• 5） 另外我觉得作为比对材料来说，比较两个材料之间的能量也是一种间接的比较的方法，用能量比较来去重在分子晶体里面也常用，但是一般会用非常cheap的经典力场来进行去重，你的程序里面用mlff的能量来进行比较去重我觉得反而增加计算量