You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Certain rhombohedral unit cells will contain duplicate sets of points, typically in the ratio n_points/n_expected=3, with a few isolated examples of n_p/n_e ∈ {3/2, 7/6}. In the 3x case, the replicated atoms each form a complete, valid unit cell for the crystal with a different translational center but otherwise equivalent spacing: the obverse and reverse settings of the cell. In these cases, we are building a complete hexagonal structure even if the underlying crystal is more simply represented as a rhombohedral one.
Note that this is different from the issue reported in #42 in which numerical precision was the primary driver of duplicate atoms. While this also occurred in a rhombohedral crystal (due to the representation of 1/3 and 2/3 in limited precision), this underlying issue is slightly different.
It is possible to convert to a rhombohedral setting and doing so could aid in particle deduplication. However, it is not trivial to apply periodic boundaries in non-orthorhombic boxes and by changing the cell type we change the fractional coordinates (which can no longer be trivially wrapped).
For now: please let me know if anyone needs the ability to convert between rhombohedral and hexagonal settings. Until that point, I will work to improve the standard deduplication procedure.
hR10-SiC:
I believe that the non-3x ratios of atoms mentioned above arise from portions of the alternate settings being deduplicated under the standard procedure when they probably should note be.
It is useful to note that this problem is not unique to parsnip - gemmi and ASE show the same behavior with the same structures (+/- about 5 files per thousand that happen to break a uniqueness check).
Work is in progress for a new approach that solves this problem - this issue is here for clarity and progress updates!
Description
Certain rhombohedral unit cells will contain duplicate sets of points, typically in the ratio
n_points/n_expected=3
, with a few isolated examples ofn_p/n_e ∈ {3/2, 7/6}
. In the 3x case, the replicated atoms each form a complete, valid unit cell for the crystal with a different translational center but otherwise equivalent spacing: the obverse and reverse settings of the cell. In these cases, we are building a complete hexagonal structure even if the underlying crystal is more simply represented as a rhombohedral one.Note that this is different from the issue reported in #42 in which numerical precision was the primary driver of duplicate atoms. While this also occurred in a rhombohedral crystal (due to the representation of
1/3
and2/3
in limited precision), this underlying issue is slightly different.It is possible to convert to a rhombohedral setting and doing so could aid in particle deduplication. However, it is not trivial to apply periodic boundaries in non-orthorhombic boxes and by changing the cell type we change the fractional coordinates (which can no longer be trivially wrapped).
For now: please let me know if anyone needs the ability to convert between rhombohedral and hexagonal settings. Until that point, I will work to improve the standard deduplication procedure.
hR10-SiC:
I believe that the non-3x ratios of atoms mentioned above arise from portions of the alternate settings being deduplicated under the standard procedure when they probably should note be.
It is useful to note that this problem is not unique to
parsnip
- gemmi and ASE show the same behavior with the same structures (+/- about 5 files per thousand that happen to break a uniqueness check).Work is in progress for a new approach that solves this problem - this issue is here for clarity and progress updates!
To reproduce
>>> CifFile("KNO3_hR8.cif").build_unit_cell().shape (24, 3) # Should be (8, 3) in the rhombohedral setting
The text was updated successfully, but these errors were encountered: