Merge pull request #150 from ThoughtWorksInc/intransitiveMap

Atry · web-flow · commit 72957b567adb · 2017-08-13T21:09:31.000-05:00
Use intransitiveMap instead of map
diff --git a/DeepLearning/src/main/scala/com/thoughtworks/deeplearning/DeepLearning.scala b/DeepLearning/src/main/scala/com/thoughtworks/deeplearning/DeepLearning.scala
@@ -62,7 +62,7 @@ trait DeepLearning[Differentiable] extends SimulacrumIssue82WorkAround[Different
 
   final def train(differentiable: Differentiable)(implicit monoid: MultiplicativeMonoid[Delta]): Future[Data] = {
     val doData = forward(differentiable).flatMap[Data] { tape =>
-      Do.garbageCollected(tape.backward(Do.now(monoid.one))).map { loss =>
+      Do.garbageCollected(tape.backward(Do.now(monoid.one))).intransitiveMap { _: Unit =>
         tape.data
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ trait DeepLearning[Differentiable] extends SimulacrumIssue82WorkAround[Different`
`62`	`62`
`63`	`63`	`final def train(differentiable: Differentiable)(implicit monoid: MultiplicativeMonoid[Delta]): Future[Data] = {`
`64`	`64`	`val doData = forward(differentiable).flatMap[Data] { tape =>`
`65`		`- Do.garbageCollected(tape.backward(Do.now(monoid.one))).map { loss =>`
	`65`	`+ Do.garbageCollected(tape.backward(Do.now(monoid.one))).intransitiveMap { _: Unit =>`
`66`	`66`	`tape.data`
`67`	`67`	`}`
`68`	`68`	`}`