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

Partially solves #229 #235

Merged
merged 1 commit into from
Jan 30, 2019

Conversation

VVitale
Copy link
Collaborator

@VVitale VVitale commented Jan 24, 2019

Apart from the small changes to the version number and date in various files, I have added some comments in the Introduction section of the User Guide to reflect the changes from the last release. In particular, I have added a paragraph in the subsection Getting Help which states that an alternative way to contact us and receive help is by raising an issue on the GitHub repository.

@codecov
Copy link

codecov bot commented Jan 24, 2019

Codecov Report

Merging #235 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #235   +/-   ##
========================================
  Coverage    62.01%   62.01%           
========================================
  Files           29       29           
  Lines        17161    17161           
========================================
  Hits         10643    10643           
  Misses        6518     6518
Impacted Files Coverage Δ
src/io.F90 70.34% <ø> (ø) ⬆️

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 709637c...fde7175. 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.

As there will probably be more changes to the documentation and tutorial, maybe we shouldn't add a precompiled PDF yet? See also my comment in #229 that actually suggests not to add the PDF at all in the repo.

@@ -55,8 +55,12 @@ example21: Gallium Arsenide - Symmetry-adapted Wannier functions

example22: Copper - Symmetry-adapted Wannier functions

example23 : Silicon - G0W0 bands structure interpolation
example23: Silicon - G0W0 bands structure interpolation
Copy link
Member

Choose a reason for hiding this comment

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

example 24 still missing

@giovannipizzi giovannipizzi merged commit 6a1f49d into wannier-developers:develop Jan 30, 2019
manxkim pushed a commit to manxkim/wannier90 that referenced this pull request Jan 10, 2021
# 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.

2 participants