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

read_hamiltonian() does not work with lanthanides? #491

Closed
novidad21 opened this issue Sep 29, 2022 · 8 comments
Closed

read_hamiltonian() does not work with lanthanides? #491

novidad21 opened this issue Sep 29, 2022 · 8 comments

Comments

@novidad21
Copy link

Describe the bug

Reproducable code

import sisl
fdf=sisl.get_sile("PATH_TO_FDF")
ham=fdf.read_hamiltonian()

Freshly installed siesta from master brench yesterday. After running a calculation with SpinOrbit true using psml pseudopotentials downloaded from dojo-pseudo.org the hamiltonian cannot be read:

ValueError: AtomicOrbital does not implement shell h and above!

Version details
3.9.12 (main, Apr 5 2022, 06:56:58)
[GCC 7.5.0]
0.0.0.dev+eacb8351ff416ad9f7f93732777c3d7931c5f711
lase.zip

@zerothi
Copy link
Owner

zerothi commented Sep 29, 2022

Yes, currently sisl does not implement the h-shell.

@zerothi
Copy link
Owner

zerothi commented Sep 29, 2022

Could you please try the latest commit? It should work now :)
Thanks!

@novidad21
Copy link
Author

Thanks @zerothi it seems the issue resolved. However I still don't really understand the issue here, because in the produced pseudopotential files I see no indication of h shells, it does not go above l=4. But anyway, no it works! Thank you.

@zerothi
Copy link
Owner

zerothi commented Sep 30, 2022

But if one of the l=4 shells gets polarized, it will be an l=5. Probably that is why. Weird anyway, I haven't had a look yet. Thanks!

@novidad21
Copy link
Author

Yes, I assumed the same, so I switched to DZ, and still I ran into the same problem.

@zerothi
Copy link
Owner

zerothi commented Sep 30, 2022

Hm. Sounds like this really needs to be checked!

@zerothi
Copy link
Owner

zerothi commented Oct 7, 2022

I'll just reopen to make me remember

@zerothi zerothi reopened this Oct 7, 2022
@zerothi
Copy link
Owner

zerothi commented Oct 10, 2022

This was related to #462. So fully fixed. Can be closed now.

@zerothi zerothi closed this as completed Oct 10, 2022
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