Skip to content

Commit

Permalink
General: Renamed space_group -> spacegroup
Browse files Browse the repository at this point in the history
  • Loading branch information
Somerandomguy10111 committed Jul 6, 2024
1 parent f99a7db commit 9c45e63
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions CrystalStructure/crystal/crystal.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ class CrystalStructure(JsonDataclass):
lengths : Lengths
angles : Angles
base : CrystalBase
space_group : Optional[int] = None
spacegroup : Optional[int] = None
volume_uc : Optional[float] = None
atomic_volume: Optional[float] = None
num_atoms : Optional[int] = None
Expand Down Expand Up @@ -61,14 +61,14 @@ def from_pymatgen(cls, pymatgen_structure: Structure) -> CrystalStructure:
def calculate_properties(self):
pymatgen_structure = self.to_pymatgen()
analyzer = SpacegroupAnalyzer(structure=pymatgen_structure, symprec=0.1, angle_tolerance=10)
self.space_group = analyzer.get_space_group_number()
self.spacegroup = analyzer.get_space_group_number()
self.volume_uc = pymatgen_structure.volume

symmetry_dataset = analyzer.get_symmetry_dataset()
self.wyckoff_symbols = symmetry_dataset['wyckoffs']
self.num_atoms = len(self.wyckoff_symbols)

pymatgen_spacegroup = SpaceGroup.from_int_number(self.space_group)
pymatgen_spacegroup = SpaceGroup.from_int_number(self.spacegroup)
self.crystal_system = pymatgen_spacegroup.crystal_system

def standardize(self):
Expand Down
2 changes: 1 addition & 1 deletion tests/t_crystal/t_properties.py
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ def test_symmetries(self):
crystal.calculate_properties()

for crystal, space_group_exp in zip(self.crystals, self.spgs):
self.assertEqual(crystal.space_group, space_group_exp)
self.assertEqual(crystal.spacegroup, space_group_exp)

expected_systems = ['orthorhombic', 'trigonal']
for crystal, system_exp in zip(self.crystals, expected_systems):
Expand Down

0 comments on commit 9c45e63

Please sign in to comment.