{"action":"created","issue":{"url":"https://api.github.com/repos/EderSantana/seya/issues/21","repository_url":"https://api.github.com/repos/EderSantana/seya","labels_url":"https://api.github.com/repos/EderSantana/seya/issues/21/labels{/name}","comments_url":"https://api.github.com/repos/EderSantana/seya/issues/21/comments","events_url":"https://api.github.com/repos/EderSantana/seya/issues/21/events","html_url":"https://github.com/EderSantana/seya/issues/21","id":138691291,"number":21,"title":"Bidirectional RNN Argument Issue","user":{"login":"MdAsifKhan","id":12439236,"avatar_url":"https://avatars.githubusercontent.com/u/12439236?v=3","gravatar_id":"","url":"https://api.github.com/users/MdAsifKhan","html_url":"https://github.com/MdAsifKhan","followers_url":"https://api.github.com/users/MdAsifKhan/followers","following_url":"https://api.github.com/users/MdAsifKhan/following{/other_user}","gists_url":"https://api.github.com/users/MdAsifKhan/gists{/gist_id}","starred_url":"https://api.github.com/users/MdAsifKhan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MdAsifKhan/subscriptions","organizations_url":"https://api.github.com/users/MdAsifKhan/orgs","repos_url":"https://api.github.com/users/MdAsifKhan/repos","events_url":"https://api.github.com/users/MdAsifKhan/events{/privacy}","received_events_url":"https://api.github.com/users/MdAsifKhan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-03-05T15:35:40Z","updated_at":"2016-03-28T06:05:53Z","closed_at":null,"body":"Hi,\r\nI am getting the following error while using bidirectional RNN with 1DCNN. \r\n\r\nerror:\r\n  File \"build/bdist.linux-x86_64/egg/keras/layers/containers.py\", line 68, in add\r\n  File \"/home/user1/keras-master/seya/seya/layers/recurrent.py\", line 55, in set_previous\r\n    self.forward.set_previous(layer, connection_map)\r\nTypeError: set_previous() takes exactly 2 arguments (3 given)\r\n\r\n\r\nMy model is:\r\n\r\nforward_lstm = LSTM(input_dim=32, output_dim=32, return_sequences=True)\r\nbackward_lstm = LSTM(input_dim=32, output_dim=32, return_sequences=True)\r\nbrnn = Bidirectional(forward=forward_lstm, backward=backward_lstm, return_sequences=True)  \r\n    model = Sequential()\r\n    model.add(Convolution1D(input_dim=10,\r\n                        input_length=100,\r\n                        nb_filter=32,\r\n                        filter_length=7,\r\n                        border_mode=\"valid\",\r\n                        activation=\"relu\",\r\n                        subsample_length=1))\r\n    model.add(MaxPooling1D(pool_length=3, stride=3))\r\n    model.add(Dropout(0.2))\r\n    model.add(brnn))\r\n    model.add(Dropout(0.5))\r\n    model.add(Flatten())\r\n    model.add(Dense(output_dim=128))\r\n    model.add(Activation('relu'))\r\n    model.add(Dense(output_dim=1))\r\n    model.add(Activation('sigmoid'))\r\n\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/EderSantana/seya/issues/comments/202248920","html_url":"https://github.com/EderSantana/seya/issues/21#issuecomment-202248920","issue_url":"https://api.github.com/repos/EderSantana/seya/issues/21","id":202248920,"user":{"login":"zzukun","id":8072908,"avatar_url":"https://avatars.githubusercontent.com/u/8072908?v=3","gravatar_id":"","url":"https://api.github.com/users/zzukun","html_url":"https://github.com/zzukun","followers_url":"https://api.github.com/users/zzukun/followers","following_url":"https://api.github.com/users/zzukun/following{/other_user}","gists_url":"https://api.github.com/users/zzukun/gists{/gist_id}","starred_url":"https://api.github.com/users/zzukun/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zzukun/subscriptions","organizations_url":"https://api.github.com/users/zzukun/orgs","repos_url":"https://api.github.com/users/zzukun/repos","events_url":"https://api.github.com/users/zzukun/events{/privacy}","received_events_url":"https://api.github.com/users/zzukun/received_events","type":"User","site_admin":false},"created_at":"2016-03-28T06:05:53Z","updated_at":"2016-03-28T06:05:53Z","body":"The function 'set_previous' was modified to {def set_previous(self, layer, reset_weights=True)} in Keras,\r\nI just delete the third parameter and it works."}}