add test for reading Hamiltonian from wannier90 output

[nils-wittemeier]

Add test for the reading of Wannier90 Hamiltonian in sisl. See related PR in sisl-files repo (sisl-files #11).

[zerothi]

Once the other PR is merged, then you should bump the files submodule in this PR as well. :)

Thanks!

[nils-wittemeier]

bump the files submodule in this PR as well.

I don't think I understand what you mean by this.

[zerothi]

The files folder is a submodule (git-terminology).
So you should do something like this:

cd <root sisl folder>
git submodule init # if files doesn't already exist and is populated
cd files
git pull
cd ..
git add files

[zerothi]

think of the files submodule as a regular file in the repo

[nils-wittemeier]

I am not sure, why this is showing a merge conflict now.

Maybe I went wrong in updating the submodule?

[zerothi]

I'll amend to this branch to try and fix it... I can't immediately see what went wrong, I'll ping you, thanks!

[zerothi]

It was because your branch was way behind main, then the submodule thing has problems. A simple rebase solved the problem.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.82%. Comparing base (53aaa64) to head (0568c12).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #723      +/-   ##
==========================================
+ Coverage   86.64%   86.82%   +0.17%     
==========================================
  Files         399      399              
  Lines       50816    50823       +7     
==========================================
+ Hits        44030    44125      +95     
+ Misses       6786     6698      -88     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.