Merge branch 'master' of github.com:MikeInnes/Flux.jl

src/tracker/Tracker.jl

@@ -45,8 +45,7 @@ tovec(xs::AbstractArray) = vec(xs)
 tovec(xs) = xs
 
 function back!(x::TrackedArray, Δ)
-  Δ′ = vec(x.Δ)
-  Δ′ .+= tovec(Δ)
+  x.Δ .+= Δ
   back!(x.f, Δ)
 end
 

src/tracker/lib.jl

@@ -68,9 +68,11 @@ function tracked_broadcast(f, args::Vararg{Any,N}) where N
   TrackedArray(Call(b, args...), b())
 end
 
+trim(x, Δ) = reshape(Δ, ntuple(i -> size(Δ, i), Val{ndims(x)}))
+
 unbroadcast(x, Δ) =
   size(x) == size(Δ) ? Δ :
-    sum(Δ, filter(n -> size(x, n) == 1, 1:ndims(Δ)))
+    trim(x, sum(Δ, filter(n -> size(x, n) == 1, 1:ndims(Δ))))
 
 function getpartial(Δ, x, i)
   @inbounds p = getindex(partials(x), i)

src/tracker/numeric.jl

@@ -1,6 +1,6 @@
 function gradient(f, xs::AbstractArray...)
   xs = track.(xs)
-  back!(f(xs...), [1])
+  back!(f(xs...))
   grad.(xs)
 end