Skip to content

Allow selection of reference charges#100

Merged
awvwgk merged 4 commits intodftd4:mainfrom
awvwgk:ref-charge
May 10, 2021
Merged

Allow selection of reference charges#100
awvwgk merged 4 commits intodftd4:mainfrom
awvwgk:ref-charge

Conversation

@awvwgk
Copy link
Copy Markdown
Member

@awvwgk awvwgk commented May 10, 2021

Required for self-consistent DFT-D4 calculations with GFN2-xTB.

@codecov
Copy link
Copy Markdown

codecov bot commented May 10, 2021

Codecov Report

Merging #100 (d7fa9ad) into main (a27f277) will decrease coverage by 0.04%.
The diff coverage is 57.25%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #100      +/-   ##
==========================================
- Coverage   60.05%   60.01%   -0.05%     
==========================================
  Files          39       39              
  Lines        2779     2886     +107     
  Branches      932      979      +47     
==========================================
+ Hits         1669     1732      +63     
- Misses        571      599      +28     
- Partials      539      555      +16     
Impacted Files Coverage Δ
src/dftd4/reference.f90 54.32% <43.75%> (+0.08%) ⬆️
test/unit/test_model.f90 55.91% <54.05%> (-1.23%) ⬇️
src/dftd4/model.f90 79.03% <94.44%> (+2.04%) ⬆️

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 a27f277...d7fa9ad. Read the comment docs.

@awvwgk awvwgk merged commit 22206af into dftd4:main May 10, 2021
@awvwgk awvwgk deleted the ref-charge branch May 10, 2021 21:16
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.

1 participant