Skip to content

Commit 6e4b4fc

Browse files
committed
discriminate virials labels in Multisystem class
1 parent c0bb798 commit 6e4b4fc

File tree

3 files changed

+16
-14
lines changed

3 files changed

+16
-14
lines changed

dpdata/system.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1154,6 +1154,8 @@ def __append(self, system):
11541154
return
11551155
self.check_atom_names(system)
11561156
formula = system.formula
1157+
if 'virials' in system.data:
1158+
formula = formula + "_virials"
11571159
if formula in self.systems:
11581160
self.systems[formula].append(system)
11591161
else:

tests/test_ase_traj.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
class TestASEtraj1(unittest.TestCase, CompLabeledSys, IsPBC):
1515
def setUp (self) :
1616
self.multi_systems = dpdata.MultiSystems.from_file('ase_traj/HeAlO.traj', fmt='ase_traj/structure')
17-
self.system_1 = self.multi_systems.systems['Al0He4O0']
17+
self.system_1 = self.multi_systems.systems['Al0He4O0_virials']
1818
self.system_2 = dpdata.LabeledSystem('ase_traj/Al0He4O0', fmt='deepmd')
1919
self.places = 6
2020
self.e_places = 6
@@ -25,11 +25,11 @@ def setUp (self) :
2525
class TestASEtraj1(unittest.TestCase, CompLabeledSys, IsPBC):
2626
def setUp (self) :
2727
self.system_temp0 = dpdata.MultiSystems.from_file(file_name='ase_traj/HeAlO.traj', fmt='ase/structure')
28-
self.system_1 = self.system_temp0.systems['Al2He1O3'] # .sort_atom_types()
28+
self.system_1 = self.system_temp0.systems['Al2He1O3_virials'] # .sort_atom_types()
2929
self.system_temp1 = dpdata.LabeledSystem('ase_traj/Al2He1O3', fmt='deepmd')
3030
self.system_temp2 = dpdata.LabeledSystem('ase_traj/Al4He4O6', fmt='deepmd')
3131
self.system_temp3 = dpdata.MultiSystems(self.system_temp2, self.system_temp1)
32-
self.system_2 = self.system_temp3.systems['Al2He1O3']
32+
self.system_2 = self.system_temp3.systems['Al2He1O3_virials']
3333
self.places = 6
3434
self.e_places = 6
3535
self.f_places = 6

tests/test_quip_gap_xyz.py

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
class TestQuipGapxyz1(unittest.TestCase, CompLabeledSys, IsPBC):
88
def setUp (self) :
99
self.multi_systems = dpdata.MultiSystems.from_file('xyz/xyz_unittest.xyz','quip/gap/xyz')
10-
self.system_1 = self.multi_systems.systems['B1C9']
10+
self.system_1 = self.multi_systems.systems['B1C9_virials']
1111
self.system_2 = dpdata.LabeledSystem('xyz/B1C9', fmt='deepmd')
1212
self.places = 6
1313
self.e_places = 6
@@ -17,11 +17,11 @@ def setUp (self) :
1717
class TestQuipGapxyz2(unittest.TestCase, CompLabeledSys, IsPBC):
1818
def setUp (self) :
1919
self.system_temp0 = dpdata.MultiSystems.from_file(file_name='xyz/xyz_unittest.xyz', fmt='quip/gap/xyz')
20-
self.system_1 = self.system_temp0.systems['B5C7'] # .sort_atom_types()
20+
self.system_1 = self.system_temp0.systems['B5C7_virials'] # .sort_atom_types()
2121
self.system_temp1 = dpdata.LabeledSystem('xyz/B1C9', fmt='deepmd')
2222
self.system_temp2 = dpdata.LabeledSystem('xyz/B5C7', fmt='deepmd')
2323
self.system_temp3 = dpdata.MultiSystems(self.system_temp2, self.system_temp1)
24-
self.system_2 = self.system_temp3.systems['B5C7']
24+
self.system_2 = self.system_temp3.systems['B5C7_virials']
2525
self.places = 6
2626
self.e_places = 6
2727
self.f_places = 6
@@ -30,10 +30,10 @@ def setUp (self) :
3030
class TestQuipGapxyzsort1(unittest.TestCase, CompLabeledSys, IsPBC):
3131
def setUp (self) :
3232
self.multi_systems_1 = dpdata.MultiSystems.from_file('xyz/xyz_unittest.sort.xyz','quip/gap/xyz')
33-
self.system_1 = self.multi_systems_1.systems['B5C7']
33+
self.system_1 = self.multi_systems_1.systems['B5C7_virials']
3434
self.system_1.sort_atom_types()
3535
self.multi_systems_2 = dpdata.MultiSystems.from_file('xyz/xyz_unittest.xyz','quip/gap/xyz')
36-
self.system_2 = self.multi_systems_2.systems['B5C7']
36+
self.system_2 = self.multi_systems_2.systems['B5C7_virials']
3737
self.places = 6
3838
self.e_places = 6
3939
self.f_places = 6
@@ -42,10 +42,10 @@ def setUp (self) :
4242
class TestQuipGapxyzsort2(unittest.TestCase, CompLabeledSys, IsPBC):
4343
def setUp (self) :
4444
self.multi_systems_1 = dpdata.MultiSystems.from_file('xyz/xyz_unittest.sort.xyz','quip/gap/xyz')
45-
self.system_1 = self.multi_systems_1.systems['B1C9']
45+
self.system_1 = self.multi_systems_1.systems['B1C9_virials']
4646
self.system_1.sort_atom_types()
4747
self.multi_systems_2 = dpdata.MultiSystems.from_file('xyz/xyz_unittest.xyz','quip/gap/xyz')
48-
self.system_2 = self.multi_systems_2.systems['B1C9']
48+
self.system_2 = self.multi_systems_2.systems['B1C9_virials']
4949
self.places = 6
5050
self.e_places = 6
5151
self.f_places = 6
@@ -54,10 +54,10 @@ def setUp (self) :
5454
class TestQuipGapxyzfield(unittest.TestCase, CompLabeledSys, IsPBC):
5555
def setUp (self) :
5656
self.multi_systems_1 = dpdata.MultiSystems.from_file('xyz/xyz_unittest.field.xyz','quip/gap/xyz')
57-
self.system_1 = self.multi_systems_1.systems['B1C9']
57+
self.system_1 = self.multi_systems_1.systems['B1C9_virials']
5858
self.system_1.sort_atom_types()
5959
self.multi_systems_2 = dpdata.MultiSystems.from_file('xyz/xyz_unittest.xyz','quip/gap/xyz')
60-
self.system_2 = self.multi_systems_2.systems['B1C9']
60+
self.system_2 = self.multi_systems_2.systems['B1C9_virials']
6161
self.places = 6
6262
self.e_places = 6
6363
self.f_places = 6
@@ -66,10 +66,10 @@ def setUp (self) :
6666
class TestQuipGapxyzfield2(unittest.TestCase, CompLabeledSys, IsPBC):
6767
def setUp (self) :
6868
self.multi_systems_1 = dpdata.MultiSystems.from_file('xyz/xyz_unittest.field.xyz','quip/gap/xyz')
69-
self.system_1 = self.multi_systems_1.systems['B5C7']
69+
self.system_1 = self.multi_systems_1.systems['B5C7_virials']
7070
self.system_1.sort_atom_types()
7171
self.multi_systems_2 = dpdata.MultiSystems.from_file('xyz/xyz_unittest.xyz','quip/gap/xyz')
72-
self.system_2 = self.multi_systems_2.systems['B5C7']
72+
self.system_2 = self.multi_systems_2.systems['B5C7_virials']
7373
self.places = 6
7474
self.e_places = 6
7575
self.f_places = 6

0 commit comments

Comments
 (0)