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
translate2uc
should use boundary conditions (?)
#764
Comments
Indeed, however, I am not sure whether it should be a default of both merged? However, I can see that the documentation specifies periodic direction, so if this is clarified in the doc-string, that |
Perhaps there are other places:
|
This goes back to the never ending discussion 😄 My view is that as long as you have periodicity, it makes sense to translate to the unit cell regardless of |
Ok, lets do that then :) |
Even more so because |
But... ;) When |
For the moment they should, right? SIESTA assumes that everything is periodic when it calculates things. But yeah when the PSolver makes it possible to not treat things periodically I can see that this will become more complicated 😅 Maybe one could check Psolver options then, I don't know. But that is a problem of how to determine boundary conditions. I think that once you have established that the boundaries are periodic I believe it makes sense to use that fact here. |
Now pbc usage has been moved across the code base (I hope everything is managed). I have added a setter for pbc to enable fast setting pbc. It will silently ignore any dimensions not specified. This may be used as a shorthand for lattice.set_boundary_condition(...) Signed-off-by: Nick Papior <nickpapior@gmail.com>
Now pbc usage has been moved across the code base (I hope everything is managed). I have added a setter for pbc to enable fast setting pbc. It will silently ignore any dimensions not specified. This may be used as a shorthand for lattice.set_boundary_condition(...) Signed-off-by: Nick Papior <nickpapior@gmail.com>
fixed pbc usage across sisl, fixes #764
Here in
Geometry.translate2uc
,nsc
is used to understand which axes to translate:sisl/src/sisl/_core/geometry.py
Line 1741 in 8d6da59
I think it should check for periodic boundary conditions instead, no?
The text was updated successfully, but these errors were encountered: