Closed tpapp closed 5 years ago
Merging #52 into master will increase coverage by
0.1%
. The diff coverage is97.5%
.
@@ Coverage Diff @@
## master #52 +/- ##
=========================================
+ Coverage 91.35% 91.46% +0.1%
=========================================
Files 6 6
Lines 162 164 +2
=========================================
+ Hits 148 150 +2
Misses 14 14
Impacted Files | Coverage Δ | |
---|---|---|
src/utilities.jl | 100% <ø> (ø) |
:arrow_up: |
src/scalar.jl | 83.78% <100%> (+0.45%) |
:arrow_up: |
src/custom.jl | 82.35% <100%> (+1.1%) |
:arrow_up: |
src/aggregation.jl | 95.65% <100%> (-0.43%) |
:arrow_down: |
src/generic.jl | 94.44% <100%> (+2.77%) |
:arrow_up: |
src/special_arrays.jl | 93.93% <83.33%> (ø) |
:arrow_up: |
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 c411b03...3583fd4. Read the comment docs.
make transform_with and index_at! do the index counting,
generalize RealVector to AbstractVector
TODO
[ ] wait for a new release of Zygote that contains https://github.com/FluxML/Zygote.jl/pull/297,
[ ] figure out why inference breaks with Zygote, does it come from LogDensityProblems or TransformVariables?
[x] make AD test transformation include all transformation types