Skip to content

Conversation

cnegre
Copy link
Collaborator

@cnegre cnegre commented Nov 25, 2019

Cleanup of the test main driver.

mtx matrix are read using bml_read_matrix


This change is Reviewable

@cnegre cnegre requested a review from nicolasbock November 25, 2019 21:21
@cnegre cnegre requested a review from suemni as a code owner November 25, 2019 21:21
@codecov
Copy link

codecov bot commented Nov 25, 2019

Codecov Report

Merging #141 into master will increase coverage by 6.7%.
The diff coverage is 71.73%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #141     +/-   ##
=========================================
+ Coverage   33.49%   40.19%   +6.7%     
=========================================
  Files          62       61      -1     
  Lines       11951    12167    +216     
  Branches     1308     1338     +30     
=========================================
+ Hits         4003     4891    +888     
+ Misses       7630     6741    -889     
- Partials      318      535    +217
Impacted Files Coverage Δ
tests/src/main.F90 80.06% <71.73%> (-2.31%) ⬇️
examples/gpmd/gpmd.F90 61.47% <0%> (-3.02%) ⬇️
examples/gpdist/gpmd_dist.F90 0% <0%> (ø) ⬆️
examples/mdresponse/mdresponse.F90 0% <0%> (ø) ⬆️
examples/gpdist/gpscf_dist.F90 0% <0%> (ø) ⬆️
examples/gpscfcov/gpscfcov.F90 0% <0%> (ø) ⬆️
examples/Mtest/metis_partition.F90 0% <0%> (ø) ⬆️
examples/gploop/gploop.F90 0% <0%> (ø) ⬆️
src/prg_densitymatrix_mod.F90 64.45% <0%> (+0.47%) ⬆️
... and 13 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 aa8f173...e1d7eb5. Read the comment docs.

@cnegre cnegre requested a review from tokshgithub December 5, 2019 17:25
@cnegre cnegre merged commit df5e364 into master Dec 11, 2019
@cnegre cnegre deleted the cleanup branch June 16, 2021 04:19
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