Skip to content
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

Centering argument for generate of non-primitive systems #33

Open
thchr opened this issue Oct 27, 2021 · 3 comments
Open

Centering argument for generate of non-primitive systems #33

thchr opened this issue Oct 27, 2021 · 3 comments

Comments

@thchr
Copy link
Owner

thchr commented Oct 27, 2021

If we add a two-fold rotation 2₁₀₀ = S"x,-y,-z" to space group 148, we should get space group 166. Instead, generate overflows:

julia> generate([spacegroup(148)..., S"x,-y,-z"]
ERROR: OverflowError: The provided set of generators overflowed Nmax distinct operations: generators may not form a finite group; otherwise, try increasing Nmax

Probably related to the centering translation of the rhombohedral lattice since the primitivized variant works correctly:

julia> generate([primitivize(spacegroup(148))..., primitivize(S"x,y,-z", 'R')])
GenericGroup{3} with 12 operations:
 1
 3₁₁₁⁺
 3₁₁₁⁻
 -1
 -3₁₁₁⁺
 -3₁₁₁⁻
 m₁₁₁
 -6₁₁₁⁻
 -6₁₁₁⁺
 2₁₁₁
 6₁₁₁⁻
 6₁₁₁⁺
@thchr
Copy link
Owner Author

thchr commented Oct 27, 2021

Actually, scratch that. The problem was that I was not using the right operator. Should've been 2₀₀₁ not 2₁₀₀...

@thchr thchr closed this as completed Oct 27, 2021
@thchr
Copy link
Owner Author

thchr commented Oct 27, 2021

... Maybe generate should take an optional centering input argument though. Otherwise the modτ argument isn't really doing quite the right thing for non-primitive lattices.

@thchr
Copy link
Owner Author

thchr commented Oct 27, 2021

Let's keep this open to investigate later.

@thchr thchr reopened this Oct 27, 2021
@thchr thchr changed the title Bug in generate for rhombohedral systems Centering argument for generate of non-primitive systems Apr 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant