-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Get cell #186
Get cell #186
Changes from all commits
cdb7771
df73b02
e29d1d8
09fa362
e2d0846
ac7e157
bc476bc
4c5834c
adc184b
99a28ac
e21bb2f
98c62c6
f82327e
0d3a426
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -51,6 +51,7 @@ | |
get_wrapped_coordinates, | ||
pymatgen_to_ase, | ||
select_index, | ||
get_cell, | ||
) | ||
|
||
# Visualize | ||
|
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -2,7 +2,7 @@ | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
from ase.atoms import Atoms | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
from ase.data import atomic_numbers | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
from scipy.sparse import coo_matrix | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
from typing import Optional | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
from typing import Optional, Union | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
def get_extended_positions( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
@@ -226,3 +226,36 @@ def apply_strain( | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
structure_copy.set_cell(cell, scale_atoms=True) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
if return_box: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
return structure_copy | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
def get_cell(cell: Union[Atoms, list, tuple, np.ndarray, float]): | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
""" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Get cell of an ase structure, or convert a float or a (3,)-array into a | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
orthogonal cell. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Args: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
cell (Atoms|ndarray|list|float|tuple): Cell | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Returns: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
(3, 3)-array: Cell | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
""" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
if isinstance(cell, Atoms): | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
return cell.cell | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
# Convert float into (3,)-array. No effect if it is (3,3)-array or | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
# (3,)-array. Raises error if the shape is not correct | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
try: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
cell = cell * np.ones(3) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
except ValueError: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
raise ValueError( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
f"Invalid cell type or shape: {type(cell).__name__}, {np.shape(cell)}" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
if np.shape(cell) == (3, 3): | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
return cell | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
# Convert (3,)-array into (3,3)-array. Raises error if the shape is wrong | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
try: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
return cell * np.eye(3) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
except ValueError: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
raise ValueError( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
f"Invalid cell type or shape: {type(cell).__name__}, {np.shape(cell)}" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Comment on lines
+231
to
+261
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Refactor 247- cell = cell * np.ones(3)
+ cell = np.full(3, cell) Consider using Committable suggestion
Suggested change
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
# coding: utf-8 | ||
# Copyright (c) Max-Planck-Institut für Eisenforschung GmbH - Computational Materials Design (CM) Department | ||
# Distributed under the terms of "New BSD License", see the LICENSE file. | ||
|
||
import unittest | ||
import numpy as np | ||
from ase.build import bulk | ||
import structuretoolkit as stk | ||
|
||
|
||
class TestHelpers(unittest.TestCase): | ||
def test_get_cell(self): | ||
self.assertEqual((3 * np.eye(3)).tolist(), stk.get_cell(3).tolist()) | ||
self.assertEqual( | ||
([1, 2, 3] * np.eye(3)).tolist(), stk.get_cell([1, 2, 3]).tolist() | ||
) | ||
atoms = bulk("Fe") | ||
self.assertEqual( | ||
atoms.cell.tolist(), stk.get_cell(atoms).tolist() | ||
) | ||
with self.assertRaises(ValueError): | ||
stk.get_cell(np.arange(4)) | ||
with self.assertRaises(ValueError): | ||
stk.get_cell(np.ones((4, 3))) | ||
|
||
|
||
if __name__ == "__main__": | ||
unittest.main() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove unused import to clean up the code.
- from ase.data import atomic_numbers
This import is unused and should be removed to clean up the codebase.
Committable suggestion