Refactor models, clean the code and separate training from testing.

1 files changed, 21 insertions, 55 deletions

diff --git a/model/time_simple_mlp.py b/model/time_simple_mlp.py
index 9e1b10a..0ce6b29 100644
--- a/model/time_simple_mlp.py
+++ b/model/time_simple_mlp.py
@@ -1,63 +1,29 @@
-from blocks.bricks import MLP, Rectifier, Linear, Sigmoid, Identity
-from blocks.bricks.lookup import LookupTable
+from blocks.bricks import application, Identity
 
-from theano import tensor
-
-import data
 import error
+from model.mlp import FFMLP, Stream
 
-class Model(object):
-    def __init__(self, config):
-        # The input and the targets
-        x_firstk_latitude = (tensor.matrix('first_k_latitude') - data.train_gps_mean[0]) / data.train_gps_std[0]
-        x_firstk_longitude = (tensor.matrix('first_k_longitude') - data.train_gps_mean[1]) / data.train_gps_std[1]
-
-        x_lastk_latitude = (tensor.matrix('last_k_latitude') - data.train_gps_mean[0]) / data.train_gps_std[0]
-        x_lastk_longitude = (tensor.matrix('last_k_longitude') - data.train_gps_mean[1]) / data.train_gps_std[1]
-
-        input_list = [x_firstk_latitude, x_firstk_longitude, x_lastk_latitude, x_lastk_longitude]
-        embed_tables = []
-
-        self.require_inputs = ['first_k_latitude', 'first_k_longitude', 'last_k_latitude', 'last_k_longitude']
-
-        for (varname, num, dim) in config.dim_embeddings:
-            self.require_inputs.append(varname)
-            vardata = tensor.lvector(varname)
-            tbl = LookupTable(length=num, dim=dim, name='%s_lookup'%varname)
-            embed_tables.append(tbl)
-            input_list.append(tbl.apply(vardata))
-
-        y = tensor.lvector('travel_time')
-
-        # Define the model
-        mlp = MLP(activations=[Rectifier() for _ in config.dim_hidden] + [Identity()],
-                           dims=[config.dim_input] + config.dim_hidden + [config.dim_output])
-
-        # Create the Theano variables
-        inputs = tensor.concatenate(input_list, axis=1)
-        # inputs = theano.printing.Print("inputs")(inputs)
-        outputs = config.exp_base ** mlp.apply(inputs)
-
-        # outputs = theano.printing.Print("outputs")(outputs)
-        # y = theano.printing.Print("y")(y)
 
-        outputs.name = 'outputs'
+class Model(FFMLP):
+    def __init__(self, config, **kwargs):
+        super(Model, self).__init__(config, output_layer=Identity, **kwargs)
+        self.inputs.append('input_time')
 
-        # Calculate the cost
-        cost = error.rmsle(outputs.flatten(), y.flatten())
-        cost.name = 'cost'
+    @application(outputs=['duration'])
+    def predict(self, **kwargs):
+        outputs = super(Model, self).predict(**kwargs)
+        return kwargs['input_time'] + self.config.exp_base ** outputs
 
-        # Initialization
-        for tbl in embed_tables:
-            tbl.weights_init = config.embed_weights_init
-        mlp.weights_init = config.mlp_weights_init
-        mlp.biases_init = config.mlp_biases_init
+    @predict.property('inputs')
+    def predict_inputs(self):
+        return self.inputs
 
-        for tbl in embed_tables:
-            tbl.initialize()
-        mlp.initialize()
+    @application(outputs=['cost'])
+    def cost(self, **kwargs):
+        y_hat = self.predict(**kwargs)
+        y = kwargs['travel_time']
+        return error.rmsle(y_hat.flatten(), y.flatten())
 
-        self.cost = cost
-        self.monitor = [cost]
-        self.outputs = outputs
-        self.pred_vars = ['travel_time']
+    @cost.property('inputs')
+    def cost_inputs(self):
+        return self.inputs + ['travel_time']