Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix band path when discontinuities are present #199

Merged
merged 4 commits into from
Sep 1, 2018

Conversation

giovannipizzi
Copy link
Member

This PR fixes #195.
It generates a path that now includes also both points if there is a path discontinuity.
Path length (x axis) also now fixes a (very small) discrepancy that was present in the previous version (typically not even visible graphically, but important when e.g. comparing band structures).
Moreover, it generates a new file seedname_band.labelinfo.dat with information on where the high-symmetry points were located.
A test and documentation of the new input file are provided.

when there are discontinuities.
Also, supporting more than one-letter symbols for
high-symmetry kpoints.

Also creating a new file with a mapping of the points with their labels.
Might not be fully tested yet.
Also, renamed it to seedname_band.labelinfo.dat
@giovannipizzi giovannipizzi requested a review from jryates August 29, 2018 21:29
@codecov
Copy link

codecov bot commented Aug 29, 2018

Codecov Report

Merging #199 into develop will increase coverage by 0.12%.
The diff coverage is 96.66%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #199      +/-   ##
===========================================
+ Coverage    60.35%   60.47%   +0.12%     
===========================================
  Files           29       29              
  Lines        16586    16629      +43     
===========================================
+ Hits         10010    10057      +47     
+ Misses        6576     6572       -4
Impacted Files Coverage Δ
src/parameters.F90 80.06% <100%> (ø) ⬆️
src/plot.F90 49.67% <96.42%> (+2.86%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f373db1...e5389ab. Read the comment docs.

@jryates jryates merged commit 086666e into wannier-developers:develop Sep 1, 2018
@giovannipizzi giovannipizzi deleted the fix_195_kpath branch September 17, 2018 17:06
manxkim pushed a commit to manxkim/wannier90 that referenced this pull request Jan 10, 2021
…kpath

Fix band path when discontinuities are present
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

missing k-point for discontinuous paths
2 participants