{"action":"created","issue":{"url":"https://api.github.com/repos/mlpack/mlpack/issues/1327","repository_url":"https://api.github.com/repos/mlpack/mlpack","labels_url":"https://api.github.com/repos/mlpack/mlpack/issues/1327/labels{/name}","comments_url":"https://api.github.com/repos/mlpack/mlpack/issues/1327/comments","events_url":"https://api.github.com/repos/mlpack/mlpack/issues/1327/events","html_url":"https://github.com/mlpack/mlpack/pull/1327","id":307216687,"number":1327,"title":"Update backward step.","user":{"login":"zoq","id":4209744,"avatar_url":"https://avatars3.githubusercontent.com/u/4209744?v=4","gravatar_id":"","url":"https://api.github.com/users/zoq","html_url":"https://github.com/zoq","followers_url":"https://api.github.com/users/zoq/followers","following_url":"https://api.github.com/users/zoq/following{/other_user}","gists_url":"https://api.github.com/users/zoq/gists{/gist_id}","starred_url":"https://api.github.com/users/zoq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zoq/subscriptions","organizations_url":"https://api.github.com/users/zoq/orgs","repos_url":"https://api.github.com/users/zoq/repos","events_url":"https://api.github.com/users/zoq/events{/privacy}","received_events_url":"https://api.github.com/users/zoq/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2018-03-21T12:01:15Z","updated_at":"2018-03-23T17:08:39Z","closed_at":"2018-03-23T15:17:26Z","author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/mlpack/mlpack/pulls/1327","html_url":"https://github.com/mlpack/mlpack/pull/1327","diff_url":"https://github.com/mlpack/mlpack/pull/1327.diff","patch_url":"https://github.com/mlpack/mlpack/pull/1327.patch"},"body":"Updated the backward step formula for the Batch Normalization layer, this should pass the Gradient Check test:\r\n- http://masterblaster.mlpack.org/view/docker/job/docker%20mlpack%20nightly%20build/armadillo_version=armadillo-7.100.3,boost_version=boost_1_49_0,buildmode=debug,compiler_version=gcc-7.2.0,label=linux-amd64/\r\n\r\n@Manthan-R-Sheth let me know what you think."},"comment":{"url":"https://api.github.com/repos/mlpack/mlpack/issues/comments/375735761","html_url":"https://github.com/mlpack/mlpack/pull/1327#issuecomment-375735761","issue_url":"https://api.github.com/repos/mlpack/mlpack/issues/1327","id":375735761,"user":{"login":"zoq","id":4209744,"avatar_url":"https://avatars3.githubusercontent.com/u/4209744?v=4","gravatar_id":"","url":"https://api.github.com/users/zoq","html_url":"https://github.com/zoq","followers_url":"https://api.github.com/users/zoq/followers","following_url":"https://api.github.com/users/zoq/following{/other_user}","gists_url":"https://api.github.com/users/zoq/gists{/gist_id}","starred_url":"https://api.github.com/users/zoq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zoq/subscriptions","organizations_url":"https://api.github.com/users/zoq/orgs","repos_url":"https://api.github.com/users/zoq/repos","events_url":"https://api.github.com/users/zoq/events{/privacy}","received_events_url":"https://api.github.com/users/zoq/received_events","type":"User","site_admin":false},"created_at":"2018-03-23T17:08:39Z","updated_at":"2018-03-23T17:08:39Z","author_association":"OWNER","body":"I don't think there is a reason to call ``Backward()`` without calling ``Forward()`` first unless I missed something?\r\n\r\nWe could still test the output of ``Backward()``, but we would have to call ``Forward`` first, the reason I removed the test was that the GradientCheck already checks the Backward pass and I don't think the output we compared against was correct."}}
{"actor":{"display_login":"zoq"}}