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

Automated wannierization #198

Draft
wants to merge 29 commits into
base: develop
Choose a base branch
from
Draft

Automated wannierization #198

wants to merge 29 commits into from

Conversation

elinscott
Copy link
Collaborator

@elinscott elinscott commented Feb 23, 2023

This PR enables automated Wannierization. Specifically, we can now...

  • perform Wannierization using projectability instead of providing explicit projections
  • disentangle blocks of bands using parallel transport using the WannierJL code

Before merging:

…e wannierize workflow for the splitting of the occ and empty manifolds wih wjl
…points; more helpful warning when having issues woth converting str to BandPath
@elinscott elinscott changed the base branch from master to develop April 28, 2023 12:51
@elinscott elinscott added the enhancement New feature or request label May 11, 2023
@pep8speaks
Copy link

pep8speaks commented Dec 14, 2023

Hello @elinscott! Thanks for updating this PR.

Line 153:23: E226 missing whitespace around arithmetic operator

Line 181:121: E501 line too long (194 > 120 characters)

Line 146:121: E501 line too long (144 > 120 characters)
Line 376:66: E226 missing whitespace around arithmetic operator

Line 133:121: E501 line too long (125 > 120 characters)
Line 180:1: E302 expected 2 blank lines, found 1

Line 26:121: E501 line too long (123 > 120 characters)
Line 507:121: E501 line too long (129 > 120 characters)
Line 669:105: E226 missing whitespace around arithmetic operator
Line 700:121: E501 line too long (159 > 120 characters)

Line 78:80: E712 comparison to True should be 'if cond is True:' or 'if cond:'
Line 99:121: E501 line too long (121 > 120 characters)
Line 112:58: E226 missing whitespace around arithmetic operator
Line 368:36: E226 missing whitespace around arithmetic operator

Line 50:9: E722 do not use bare 'except'

Line 104:47: E226 missing whitespace around arithmetic operator
Line 194:121: E501 line too long (144 > 120 characters)
Line 262:121: E501 line too long (128 > 120 characters)
Line 335:121: E501 line too long (121 > 120 characters)
Line 452:121: E501 line too long (127 > 120 characters)
Line 600:121: E501 line too long (139 > 120 characters)
Line 602:121: E501 line too long (142 > 120 characters)
Line 613:121: E501 line too long (125 > 120 characters)

Line 315:121: E501 line too long (129 > 120 characters)
Line 327:121: E501 line too long (122 > 120 characters)
Line 539:121: E501 line too long (121 > 120 characters)
Line 573:121: E501 line too long (128 > 120 characters)
Line 582:121: E501 line too long (125 > 120 characters)
Line 585:121: E501 line too long (127 > 120 characters)
Line 588:121: E501 line too long (153 > 120 characters)
Line 620:121: E501 line too long (122 > 120 characters)
Line 632:121: E501 line too long (155 > 120 characters)
Line 1456:121: E501 line too long (130 > 120 characters)

Line 27:22: E226 missing whitespace around arithmetic operator
Line 40:22: E226 missing whitespace around arithmetic operator

Comment last updated at 2023-12-22 09:55:21 UTC

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

Successfully merging this pull request may close these issues.

2 participants