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

579 dftb hamiltonian #686

Merged
merged 3 commits into from
Feb 21, 2024
Merged

579 dftb hamiltonian #686

merged 3 commits into from
Feb 21, 2024

Conversation

zerothi
Copy link
Owner

@zerothi zerothi commented Feb 21, 2024

  • Closes Reading DFTB+ matrices #579
  • Added tests for new/changed functions?
  • Ran isort . and black . [24.2.0] at top-level
  • Documentation for functionality in docs/
  • Changes documented in CHANGELOG.md

Sorry, something went wrong.

Signed-off-by: Nick Papior <[email protected]>
This enables one to do transport with
DFTB and various other kinds of things.

This fixes #579.
One cannot read density matrices since
they are not written to disk.

Signed-off-by: Nick Papior <[email protected]>
Signed-off-by: Nick Papior <[email protected]>
"""
from .sile import * # isort: split
from .realdat import *

Check notice

Code scanning / CodeQL

'import *' may pollute namespace Note

Import pollutes the enclosing namespace, as the imported module
sisl.io.dftb.realdat
does not define '__all__'.

# isort: split

from .bigdft import *

Check notice

Code scanning / CodeQL

'import *' may pollute namespace Note

Import pollutes the enclosing namespace, as the imported module
sisl.io.bigdft
does not define '__all__'.
# isort: split

from .bigdft import *
from .dftb import *

Check notice

Code scanning / CodeQL

'import *' may pollute namespace Note

Import pollutes the enclosing namespace, as the imported module
sisl.io.dftb
does not define '__all__'.
@zerothi zerothi merged commit d440f07 into main Feb 21, 2024
6 checks passed
@zerothi zerothi deleted the 579-dftb-hamiltonian branch February 21, 2024 14:06
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

Successfully merging this pull request may close these issues.

Reading DFTB+ matrices
1 participant