{"action":"created","issue":{"url":"https://api.github.com/repos/dmlc/mxnet/issues/5798","repository_url":"https://api.github.com/repos/dmlc/mxnet","labels_url":"https://api.github.com/repos/dmlc/mxnet/issues/5798/labels{/name}","comments_url":"https://api.github.com/repos/dmlc/mxnet/issues/5798/comments","events_url":"https://api.github.com/repos/dmlc/mxnet/issues/5798/events","html_url":"https://github.com/dmlc/mxnet/pull/5798","id":221330825,"number":5798,"title":"Fixed broken SpatialTransformerNetwork example","user":{"login":"facundoq","id":1576711,"avatar_url":"https://avatars1.githubusercontent.com/u/1576711?v=3","gravatar_id":"","url":"https://api.github.com/users/facundoq","html_url":"https://github.com/facundoq","followers_url":"https://api.github.com/users/facundoq/followers","following_url":"https://api.github.com/users/facundoq/following{/other_user}","gists_url":"https://api.github.com/users/facundoq/gists{/gist_id}","starred_url":"https://api.github.com/users/facundoq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/facundoq/subscriptions","organizations_url":"https://api.github.com/users/facundoq/orgs","repos_url":"https://api.github.com/users/facundoq/repos","events_url":"https://api.github.com/users/facundoq/events{/privacy}","received_events_url":"https://api.github.com/users/facundoq/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2017-04-12T17:12:25Z","updated_at":"2017-04-13T14:00:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dmlc/mxnet/pulls/5798","html_url":"https://github.com/dmlc/mxnet/pull/5798","diff_url":"https://github.com/dmlc/mxnet/pull/5798.diff","patch_url":"https://github.com/dmlc/mxnet/pull/5798.patch"},"body":"Re-added option to include a SpatialTransformerNetwork layer in the lenet/mnist example, and a cmd argument in train_mnist to activate it.\r\n\r\nThe symbol \"lenet\" had a parameter to add a STN layer, but it was broken because the definition of the location network was missing (got removed in a previous commit, i'm guessing accidentally).\r\n\r\nI also added a command line argument to train_mnist.py to add the STN layer to the network.\r\n\r\n\r\n\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/dmlc/mxnet/issues/comments/293903764","html_url":"https://github.com/dmlc/mxnet/pull/5798#issuecomment-293903764","issue_url":"https://api.github.com/repos/dmlc/mxnet/issues/5798","id":293903764,"user":{"login":"facundoq","id":1576711,"avatar_url":"https://avatars1.githubusercontent.com/u/1576711?v=3","gravatar_id":"","url":"https://api.github.com/users/facundoq","html_url":"https://github.com/facundoq","followers_url":"https://api.github.com/users/facundoq/followers","following_url":"https://api.github.com/users/facundoq/following{/other_user}","gists_url":"https://api.github.com/users/facundoq/gists{/gist_id}","starred_url":"https://api.github.com/users/facundoq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/facundoq/subscriptions","organizations_url":"https://api.github.com/users/facundoq/orgs","repos_url":"https://api.github.com/users/facundoq/repos","events_url":"https://api.github.com/users/facundoq/events{/privacy}","received_events_url":"https://api.github.com/users/facundoq/received_events","type":"User","site_admin":false},"created_at":"2017-04-13T14:00:50Z","updated_at":"2017-04-13T14:00:50Z","body":"I think there are two types of \"understanding\" you may want to accommodate:\n1) How to use the API. For this you need the example to be as simple as\npossible (even if it doesn't make much sense in terms of complexity vs\naccuracy, as in the example in this pr)\n2) How to effectively use STNs: For this, sure, you may want to change the\nlocalisation network, but most important you'll need another dataset\n(vanilla mnist is too easy) .\nEl abr. 13, 2017 1:40, \"Xingjian Shi\" <notifications@github.com> escribió:\n\n> *@sxjscience* commented on this pull request.\n> ------------------------------\n>\n> In example/image-classification/symbols/lenet.py\n> <https://github.com/dmlc/mxnet/pull/5798#discussion_r111310429>:\n>\n> > @@ -5,6 +5,22 @@\n>  \"\"\"\n>  import mxnet as mx\n>\n> +def get_loc(data, attr={'lr_mult':'0.01'}):\n> +    \"\"\"\n> +    the localisation network in lenet-stn, it will increase acc about more than 1%,\n> +    when num-epoch >=15\n> +    \"\"\"\n> +    loc = mx.symbol.Convolution(data=data, num_filter=30, kernel=(5, 5), stride=(2,2))\n> +    loc = mx.symbol.Activation(data = loc, act_type='relu')\n> +    loc = mx.symbol.Pooling(data=loc, kernel=(2, 2), stride=(2, 2), pool_type='max')\n> +    loc = mx.symbol.Convolution(data=loc, num_filter=60, kernel=(3, 3), stride=(1,1), pad=(1, 1))\n> +    loc = mx.symbol.Activation(data = loc, act_type='relu')\n> +    loc = mx.symbol.Pooling(data=loc, global_pool=True, kernel=(2, 2), pool_type='avg')\n>\n> We can keep the example but we need to remove the kernel=(2,2) here\n>\n> My point is that we cannot understand STN by running this example. We need\n> a more accurate example that compare networks with/without STN.\n>\n> —\n> You are receiving this because you authored the thread.\n> Reply to this email directly, view it on GitHub\n> <https://github.com/dmlc/mxnet/pull/5798#pullrequestreview-32545954>, or mute\n> the thread\n> <https://github.com/notifications/unsubscribe-auth/ABgPB7DXS2z0ZNXNjdQ4kiPx-bWnmtBbks5rvacxgaJpZM4M7ttH>\n> .\n>\n"}}
{"actor":{"display_login":"facundoq"}}