-
Notifications
You must be signed in to change notification settings - Fork 145
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
Gyrotropic module #160
Gyrotropic module #160
Conversation
berry_uHu_formatted)
Conflicts: src/parameters.F90
latest commits of wannier-developers
Conflicts: src/parameters.F90 src/postw90/berry.F90 src/postw90/wan_ham.F90 src/utility.F90
Stepan - many thanks for this. Looks like there are a few small fortran/makefile issues to sort out. |
extra argument removed
Hi, Jonathan. Sorry for not responding for two weeks. I was out of office these days. Now I have fixed the errors, the check is passed. Regarding the tests, Fe is not really suitable as a gyrotropic testcase, so the folder wannier90/test-suite/tests/testpostw90_te_gyrotropic/ contains a test case for Te, which is a reduced version of example24. (accuracy parameters are reduced to unphysically small values). It contatins all input files, wannier files, and the output, which I get. It amounts to 2MB, which I think is not really much. Could you make an automated test out of this, as I am not experienced in this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All looks good!
Gyrotropic module
The present pull request adds a new module, called "gyrotropic", which was used for the calculations, presented in the article:
S. S. Tsirkin, P. Aguado Puente and I. Souza, arXiv:1710.03204
Changes in the code are
New file gyrotropic.F90
corresponding changes in postw90.F90, postw90_common.F90, parameters.F9
added optional parameter "occ" in subrotines:
berry_get_imf_klist
berry_get_imfgh_klist
wham_get_JJp_JJm_list
wham_get_eig_UU_HH_JJlist
wham_get_occ_mat_list (also reordered the parameters, so "eig" is now also opional)
in io.F90 changed 'maxlen' to 255
added new functions:
utility_det3
spin_get_S
example24
Corresponding contribution to the User Guide and Tutorial