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

Why SiestaBaseWorkchain doesn't shift bands to the fermi level? #84

Open
pfebrer opened this issue Nov 5, 2020 · 1 comment
Open

Why SiestaBaseWorkchain doesn't shift bands to the fermi level? #84

pfebrer opened this issue Nov 5, 2020 · 1 comment
Labels

Comments

@pfebrer
Copy link
Collaborator

pfebrer commented Nov 5, 2020

Just curious, because I thought the most typical thing was to plot the bands with the fermi level at 0 eV. And if you need to find the fermi level you need to retrieve it from the calculation, which is probably done differently for each code.

Maybe my real question is why Bandsdata doesn't store the fermi level, I don't know.

@bosonie
Copy link
Member

bosonie commented Nov 14, 2020

I did my research and there is only the option y_origin= when you plot (or export). The Fermi energy is not stored inside the BandsData.
Not always is needed to shift the bands to the Fermi energy, but I agree it might be a nice addition having the Fermi energy as optional attribute of the class. Do you want to comment here:
aiidateam/aiida-core#2847

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants