-
Notifications
You must be signed in to change notification settings - Fork 5
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
Use of generated_group
is unclear
#11
Comments
Thank you for filing the issue. I am amazed you found the function generated_group, but probably it was somewhere in the examples. This function will be deleted soon, as I think it is a bit opaque what it does and doesn't really serve a good purpose. Still, It's weird you got an error here. What I would suggest is that you explicitly build the whole group. You can actually multiply Permutations with the "*" operator and you can compute inverses, see https://awietek.github.io/xdiag/documentation/symmetries/permutation/ I would suggest that you explicitly build the group with multiplication and using inverses. One also needs to add the identity element. Could you give that a try and let me know if you still have the same issue? |
Running this code yields the log
$D_{3d}$ is a perfectly well defined group. Where is the mistake here?
Error constructing PermutationGroup: group multiplication not closed
which is clearly untrue, since
For reference, I'm running
XDiag Version: 0.2.0
Git hash : 4891db9
The text was updated successfully, but these errors were encountered: