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

Ortho test #106

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Ortho test #106

wants to merge 4 commits into from

Conversation

cnegre
Copy link
Collaborator

@cnegre cnegre commented Apr 4, 2019

Added missing lines from getforce.
Added a short test for ortho

@codecov
Copy link

codecov bot commented Apr 4, 2019

Codecov Report

Merging #106 into master will increase coverage by 0.3%.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master    #106     +/-   ##
========================================
+ Coverage    46.1%   46.4%   +0.3%     
========================================
  Files         200     200             
  Lines       16321   16323      +2     
========================================
+ Hits         7526    7576     +50     
+ Misses       8795    8747     -48
Impacted Files Coverage Δ
src/getforce.F90 90.6% <ø> (ø) ⬆️
src/bodirectprogress.F90 81.5% <100%> (+1%) ⬆️
src/kbldnewh.F90 42.5% <0%> (-0.7%) ⬇️
src/bldnewHS_sp.F90 78.6% <0%> (+1.1%) ⬆️
src/diagmyh.F90 66.6% <0%> (+4.1%) ⬆️
src/addqdep.F90 19.7% <0%> (+4.6%) ⬆️
src/readtb.F90 91% <0%> (+5.3%) ⬆️
src/getdeltaq.F90 67.9% <0%> (+5.6%) ⬆️
src/summary.F90 84.8% <0%> (+8%) ⬆️
src/readppot.F90 90% <0%> (+15%) ⬆️

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 e680a10...ee9c677. Read the comment docs.

@cnegre cnegre requested a review from suemni April 19, 2019 19:35
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