Update.

[dagnn.git] / test-dagnn.lua

diff --git a/test-dagnn.lua b/test-dagnn.lua
index a41d880..3801956 100755 (executable)
--- a/test-dagnn.lua
+++ b/test-dagnn.lua
@@ -23,9 +23,8 @@ require 'torch'
 require 'nn'
 require 'dagnn'
 
--- torch.setnumthreads(params.nbThreads)
 torch.setdefaulttensortype('torch.DoubleTensor')
-torch.manualSeed(2)
+torch.manualSeed(1)
 
 function checkGrad(model, criterion, input, target)
    local params, gradParams = model:getParameters()
@@ -39,6 +38,8 @@ function checkGrad(model, criterion, input, target)
    model:backward(input, gradOutput)
    local analyticalGradParam = gradParams:clone()
 
+   local err = 0
+
    for i = 1, params:size(1) do
       local x = params[i]
 
@@ -54,23 +55,13 @@ function checkGrad(model, criterion, input, target)
 
       local ana = analyticalGradParam[i]
       local num = (loss1 - loss0) / (2 * epsilon)
-      local err
 
-      if num == ana then
-         err = 0
-      else
-         err = torch.abs(num - ana) / torch.abs(num)
+      if num ~= ana then
+         err = math.max(err, torch.abs(num - ana) / torch.abs(num))
       end
-
-      print(
-         'CHECK '
-            .. err
-            .. ' checkGrad ' .. i
-            .. ' analytical ' .. ana
-            .. ' numerical ' .. num
-      )
    end
 
+   return err
 end
 
 function printTensorTable(t)
@@ -84,13 +75,13 @@ function printTensorTable(t)
    end
 end
 
---               +- Linear(10, 10) -> ReLU ---> d --+
---              /                              /     \
---             /                              /       \
---  --> a --> b -----------> c --------------+         e -->
---                            \                       /
---                             \                     /
---                              +-- Mul(-1) --------+
+--               +-- Linear(10, 10) --> ReLU --> d --+
+--              /                              /      \
+--             /                              /        \
+--  --> a --> b -----------> c --------------+          e -->
+--                            \                        /
+--                             \                      /
+--                              +----- Mul(-1) ------+
 
 model = nn.DAG()
 
@@ -100,12 +91,11 @@ c = nn.Linear(10, 15)
 d = nn.CMulTable()
 e = nn.CAddTable()
 
-model:addEdge(a, b)
-model:addEdge(b, nn.Linear(10, 15), nn.ReLU(), d)
-model:addEdge(d, e)
-model:addEdge(b, c)
-model:addEdge(c, d)
-model:addEdge(c, nn.Mul(-1), e)
+model:connect(a, b, c)
+model:connect(b, nn.Linear(10, 15), nn.ReLU(), d)
+model:connect(d, e)
+model:connect(c, d)
+model:connect(c, nn.Mul(-1), e)
 
 model:setInput(a)
 model:setOutput(e)
@@ -115,4 +105,7 @@ local output = model:updateOutput(input):clone()
 
 output:uniform()
 
-checkGrad(model, nn.MSECriterion(), input, output)
+print('Error = ' .. checkGrad(model, nn.MSECriterion(), input, output))
+
+print('Writing /tmp/graph.dot')
+model:saveDot('/tmp/graph.dot')