search

JuliaDiff / DiffRules.jl

A simple shared suite of common derivative definitions
Other
74 stars 38 forks source link

Add integration tests #61

Closed mcabbott closed 3 years ago

mcabbott commented 3 years ago

This should add a CI run which tests ForwardDiff. Based on https://github.com/SciML/DiffEqBase.jl/blob/master/.github/workflows/Downstream.yml

Possibly other packages should be included. There's a list here: https://juliahub.com/ui/Packages/DiffRules/x2ZNk/1.2.1?t=2

I added a few. If I couldn't find the tests, or the took over an hour to run, or they seemed to be failing already, then I skipped the package. (Nabla somehow got in despite failing here: https://github.com/invenia/Nabla.jl/runs/2981519710?check_suite_focus=true, same message a few days ago so not caused by latest DiffRules PRs.)

codecov-commenter commented 3 years ago

Codecov Report

Merging #61 (14ca2a6) into master (42b11d0) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #61   +/-   ##
=======================================
  Coverage   93.93%   93.93%           
=======================================
  Files           2        2           
  Lines         132      132           
=======================================
  Hits          124      124           
  Misses          8        8

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 42b11d0...14ca2a6. Read the comment docs.