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

Checking wannier90 process #3

Open
DHKiem opened this issue Oct 10, 2023 · 3 comments
Open

Checking wannier90 process #3

DHKiem opened this issue Oct 10, 2023 · 3 comments

Comments

@DHKiem
Copy link
Member

DHKiem commented Oct 10, 2023

it seems there exist some issues in wannier90 jx calculation.

@DHKiem
Copy link
Member Author

DHKiem commented Oct 11, 2023

The example was incompatible with wannier center conventions. example will be updated

@DHKiem DHKiem closed this as completed Oct 11, 2023
@DHKiem DHKiem reopened this Jan 12, 2024
@DHKiem
Copy link
Member Author

DHKiem commented Jan 12, 2024

guessed initial wannier centers in wannier90.win and calculated wannier center in wannier90.wout are not consistent. The atomic positions in .toml should be clarified as the Wannier centers in wannier90.wout, not the atomic positions.

@bluehope
Copy link
Member

In my simple examples, the atom center didn't move much and I didn't care, but if it did, it would make more sense from a proper Hamiltonian perspective to use the new positions.

However, the position of the center may change from orbit to orbit, which may not be what we want, so it may be better to use keepCentre of Wannier90 whenever possible. Still, if you have an example that doesn't work with this option, let's dig deeper :)

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

2 participants