-
Notifications
You must be signed in to change notification settings - Fork 17
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
workchain for spin-orbit-coupling #10
Comments
Reports from Giovanni: changing the call parameters of
This has been done in c00dd54 and 655c0f9, for |
Reports from Giovanni: moving the import inside the if statement:
Done in 30bd0f7, for |
Currently, the workchain uses pslibrary SOC pseudos, consider using the (norm-conserving) full-relativistic pseudos from the pseudo-dojo (FR v0.4, PBE, stringent)? TODO: need cutoff table |
problems with the function
Seems there are some slight differences between pseudos generated by different codes. I think we can skip these lines if there is no
For a given quantum number
E.g. for
In fact this python function is a translation of the QE Fortran code https://github.com/QEF/q-e/blob/7d5cebcf1250114756b88c6064ebe82e6f8fd835/Modules/uspp.f90#L69-L75 |
Thanks for the last clarification! It might be good to add this as a comment in the code so it's easier to understand. |
Thanks, done in a41bd77 |
Now the
Wannier90BandsWorkChain
in the opengrid branch supports spin-orbit-coupling (SOC) calculations.Some prerequisites for using the workchain:
aiida-quantumespresso
should contain at least commit aiidateam/aiida-quantumespresso@031b92bverdi data upf uploadfamily
. If you want to use other pseudos, just modify the json file.How to submit the workchain:
just add one line like this
aiida-wannier90-workflows/examples/workflows/run_automated_wannier.py
Line 187 in 1decab3
Let me know if you have comments, suggestions!
The text was updated successfully, but these errors were encountered: