Skip to content

Commit 5c8faa6

Browse files
authored
bug_fix | to support multi-element cases (#608)
currently, only count the #atoms of the first element when evaluating E-per-atom;
1 parent 33cf245 commit 5c8faa6

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

dpgen/auto_test/EOS.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -169,8 +169,8 @@ def _compute_lower(self,
169169
# vol = self.vol_start + ii * self.vol_step
170170
vol = loadfn(os.path.join(all_tasks[ii], 'eos.json'))['volume']
171171
task_result = loadfn(all_res[ii])
172-
res_data[vol] = task_result['energies'][-1] / task_result['atom_numbs'][0]
173-
ptr_data += '%7.3f %8.4f \n' % (vol, task_result['energies'][-1] / task_result['atom_numbs'][0])
172+
res_data[vol] = task_result['energies'][-1] / task_result['atom_numbs'].sum()
173+
ptr_data += '%7.3f %8.4f \n' % (vol, task_result['energies'][-1] / task_result['atom_numbs'].sum())
174174
# res_data[vol] = all_res[ii]['energy'] / len(all_res[ii]['force'])
175175
# ptr_data += '%7.3f %8.4f \n' % (vol, all_res[ii]['energy'] / len(all_res[ii]['force']))
176176

0 commit comments

Comments
 (0)