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

fix the issue in plot_bvec #337

Merged
merged 2 commits into from
Jul 17, 2020
Merged

Conversation

hjunlee
Copy link
Contributor

@hjunlee hjunlee commented Jul 12, 2020

Dear all:

This commit fixes the issue in writing *.bvec file which is necessary for the evaluation of velocity matrix elements in EPW.
Previously, the writing of bvec file is triggered by the input flags which are not directly related to it.
Additionally, I removed one of two open statements and increased the number of significant digits in *.bvec file so that it matches those from kmesh.pl .

On behalf of the EPW developers' team,
Hyungjun Lee

@codecov
Copy link

codecov bot commented Jul 12, 2020

Codecov Report

Merging #337 into develop will decrease coverage by 0.44%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #337      +/-   ##
===========================================
- Coverage    66.40%   65.96%   -0.45%     
===========================================
  Files           29       29              
  Lines        17992    17774     -218     
===========================================
- Hits         11948    11724     -224     
- Misses        6044     6050       +6     
Impacted Files Coverage Δ
src/plot.F90 49.40% <100.00%> (-0.06%) ⬇️
src/wannier_lib.F90 85.18% <100.00%> (ø)
src/postw90/geninterp.F90 69.03% <0.00%> (-5.81%) ⬇️
src/postw90/postw90_common.F90 63.17% <0.00%> (-5.69%) ⬇️
src/postw90/postw90.F90 84.33% <0.00%> (-2.57%) ⬇️
src/postw90/boltzwann.F90 71.31% <0.00%> (-1.56%) ⬇️
src/parameters.F90 81.53% <0.00%> (-1.54%) ⬇️
src/wannier_prog.F90 80.99% <0.00%> (-0.98%) ⬇️
src/postw90/kslice.F90 68.46% <0.00%> (-0.50%) ⬇️
test-suite/library-mode-test/test_library.F90 64.70% <0.00%> (-0.35%) ⬇️
... and 3 more

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 dd533de...c58ff61. Read the comment docs.

Copy link
Member

@giovannipizzi giovannipizzi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@giovannipizzi
Copy link
Member

For some reason I cannot automatically merge - maybe it's a temporary issue with GitHub. Can you merge develop in your branch? I'll merge then.

@hjunlee
Copy link
Contributor Author

hjunlee commented Jul 15, 2020

Dear Giovanni Pizzi:

As you suggested, I merged the develop in W90 in my branch.

Sincerely,

Hyungjun Lee

@giovannipizzi giovannipizzi merged commit 7ce080c into wannier-developers:develop Jul 17, 2020
manxkim pushed a commit to manxkim/wannier90 that referenced this pull request Jan 10, 2021
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.

2 participants