Skip to content

Add ReverseDiff support.#23

Merged
tpapp merged 4 commits into
masterfrom
tp/reversediff
Nov 18, 2018
Merged

Add ReverseDiff support.#23
tpapp merged 4 commits into
masterfrom
tp/reversediff

Conversation

@tpapp
Copy link
Copy Markdown
Owner

@tpapp tpapp commented Nov 18, 2018

No description provided.

@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 18, 2018

Codecov Report

Merging #23 into master will increase coverage by 12.14%.
The diff coverage is 66.66%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #23       +/-   ##
========================================
+ Coverage   72.85%   85%   +12.14%     
========================================
  Files           3     4        +1     
  Lines          70    80       +10     
========================================
+ Hits           51    68       +17     
+ Misses         19    12        -7
Impacted Files Coverage Δ
src/AD_ForwardDiff.jl 80% <0%> (+8.57%) ⬆️
src/LogDensityProblems.jl 89.13% <50%> (+8.17%) ⬆️
src/AD_ReverseDiff.jl 85.71% <85.71%> (ø)
src/AD_Flux.jl 71.42% <0%> (+42.85%) ⬆️

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 b38e0ba...b1e320e. Read the comment docs.

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 87

  • 7 of 10 (70.0%) changed or added relevant lines in 3 files are covered.
  • 3 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+10.3%) to 83.117%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/AD_ForwardDiff.jl 0 1 0.0%
src/AD_ReverseDiff.jl 6 7 85.71%
src/LogDensityProblems.jl 1 2 50.0%
Files with Coverage Reduction New Missed Lines %
src/LogDensityProblems.jl 3 86.36%
Totals Coverage Status
Change from base Build 84: 10.3%
Covered Lines: 64
Relevant Lines: 77

💛 - Coveralls

@coveralls
Copy link
Copy Markdown

coveralls commented Nov 18, 2018

Pull Request Test Coverage Report for Build 93

  • 8 of 12 (66.67%) changed or added relevant lines in 3 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+10.7%) to 83.544%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/AD_ForwardDiff.jl 0 1 0.0%
src/AD_ReverseDiff.jl 6 7 85.71%
src/LogDensityProblems.jl 2 4 50.0%
Files with Coverage Reduction New Missed Lines %
src/LogDensityProblems.jl 1 86.96%
Totals Coverage Status
Change from base Build 84: 10.7%
Covered Lines: 66
Relevant Lines: 79

💛 - Coveralls

@tpapp tpapp merged commit da39435 into master Nov 18, 2018
@tpapp tpapp deleted the tp/reversediff branch November 18, 2018 18:22
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