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

Pauli model in transport #1254

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

bhourahine
Copy link
Member

@bhourahine bhourahine commented Jun 17, 2023

Rebase and update spin-transport branch.

[x] - Update to current main code
[ ] - Transport geometry working correctly
[ ] - Debug forces in GaAs regression case against closed boundaries
[ ] - Transport case(s) for regression
[ ] - Partial densites etc. debugging

@codecov
Copy link

codecov bot commented Jun 17, 2023

Codecov Report

Patch coverage: 92.06% and project coverage change: +0.12 🎉

Comparison is base (79a491a) 69.71% compared to head (171b557) 69.83%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1254      +/-   ##
==========================================
+ Coverage   69.71%   69.83%   +0.12%     
==========================================
  Files         226      226              
  Lines       42243    42435     +192     
==========================================
+ Hits        29450    29636     +186     
- Misses      12793    12799       +6     
Impacted Files Coverage Δ
src/dftbp/dftb/shift.F90 56.70% <ø> (ø)
src/dftbp/type/parallelks.F90 88.88% <ø> (ø)
src/dftbp/dftbplus/transportio.F90 42.39% <33.33%> (+0.26%) ⬆️
src/dftbp/dftbplus/initprogram.F90 82.88% <50.00%> (-0.01%) ⬇️
src/dftbp/transport/negfint.F90 57.43% <74.50%> (+0.82%) ⬆️
src/dftbp/dftbplus/parser.F90 77.76% <90.00%> (-0.01%) ⬇️
src/dftbp/dftbplus/main.F90 90.79% <91.66%> (+0.01%) ⬆️
src/dftbp/dftb/spinorbit.F90 100.00% <100.00%> (ø)
src/dftbp/transport/matconv.F90 61.83% <100.00%> (+21.33%) ⬆️

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Add 2 tests for transport + SOC

GF solver working, but NEGF transport probably not.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants