{"action":"created","issue":{"url":"https://api.github.com/repos/dmlc/mxnet/issues/5300","repository_url":"https://api.github.com/repos/dmlc/mxnet","labels_url":"https://api.github.com/repos/dmlc/mxnet/issues/5300/labels{/name}","comments_url":"https://api.github.com/repos/dmlc/mxnet/issues/5300/comments","events_url":"https://api.github.com/repos/dmlc/mxnet/issues/5300/events","html_url":"https://github.com/dmlc/mxnet/pull/5300","id":212600900,"number":5300,"title":"Added eval(...) to Symbol class","user":{"login":"zackchase","id":2390222,"avatar_url":"https://avatars3.githubusercontent.com/u/2390222?v=3","gravatar_id":"","url":"https://api.github.com/users/zackchase","html_url":"https://github.com/zackchase","followers_url":"https://api.github.com/users/zackchase/followers","following_url":"https://api.github.com/users/zackchase/following{/other_user}","gists_url":"https://api.github.com/users/zackchase/gists{/gist_id}","starred_url":"https://api.github.com/users/zackchase/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zackchase/subscriptions","organizations_url":"https://api.github.com/users/zackchase/orgs","repos_url":"https://api.github.com/users/zackchase/repos","events_url":"https://api.github.com/users/zackchase/events{/privacy}","received_events_url":"https://api.github.com/users/zackchase/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-03-08T00:22:28Z","updated_at":"2017-03-08T04:36:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dmlc/mxnet/pulls/5300","html_url":"https://github.com/dmlc/mxnet/pull/5300","diff_url":"https://github.com/dmlc/mxnet/pull/5300.diff","patch_url":"https://github.com/dmlc/mxnet/pull/5300.patch"},"body":"Added an eval() method to the symbol class.\r\n\r\nPreviously one had to run\r\n~~~~\r\na = S.normal(shape=(2,2))\r\nd = a.bind({...})\r\ne = d.forward()\r\n~~~~\r\n\r\nNow you could just run \r\n~~~~\r\na = S.normal(shape=(2,2))\r\na.eval()\r\n~~~~\r\n\r\nNote: eval takes args `(self, args=None, ctx=cpu())`.\r\nThe order of args and ctx is flipped to allow for ctx to be given optionally.\r\nAlso, args is given as a named argument so that it can be omitted in the (rare) case that the symbol requires no placeholders to be filled.\r\n\r\nResolves Issue #5280 "},"comment":{"url":"https://api.github.com/repos/dmlc/mxnet/issues/comments/284944573","html_url":"https://github.com/dmlc/mxnet/pull/5300#issuecomment-284944573","issue_url":"https://api.github.com/repos/dmlc/mxnet/issues/5300","id":284944573,"user":{"login":"zackchase","id":2390222,"avatar_url":"https://avatars3.githubusercontent.com/u/2390222?v=3","gravatar_id":"","url":"https://api.github.com/users/zackchase","html_url":"https://github.com/zackchase","followers_url":"https://api.github.com/users/zackchase/followers","following_url":"https://api.github.com/users/zackchase/following{/other_user}","gists_url":"https://api.github.com/users/zackchase/gists{/gist_id}","starred_url":"https://api.github.com/users/zackchase/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zackchase/subscriptions","organizations_url":"https://api.github.com/users/zackchase/orgs","repos_url":"https://api.github.com/users/zackchase/repos","events_url":"https://api.github.com/users/zackchase/events{/privacy}","received_events_url":"https://api.github.com/users/zackchase/received_events","type":"User","site_admin":false},"created_at":"2017-03-08T04:36:17Z","updated_at":"2017-03-08T04:36:17Z","body":"Oh lol missed that lol. :) Was setting up my local environment today.\nThanks!\n\nOn Tue, Mar 7, 2017 at 8:34 PM, Eric Junyuan Xie <notifications@github.com>\nwrote:\n\n> *@piiswrong* commented on this pull request.\n> ------------------------------\n>\n> In python/mxnet/symbol.py\n> <https://github.com/dmlc/mxnet/pull/5300#discussion_r104842082>:\n>\n> > @@ -1025,6 +1025,36 @@ def grad(self, wrt):\n>          return Symbol(handle)\n>      # pylint: enable= no-member\n>\n> +    def eval(self, args=None, ctx=cpu()):\n> +        if args == None:\n>\n> Oh I meant removing the line print(\"hello world!\") and move these two\n> lines down to under the docstring\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/5300#discussion_r104842082>, or mute\n> the thread\n> <https://github.com/notifications/unsubscribe-auth/ACR4zvneAWOciImdVjOS3hityfHEA7hfks5rji_hgaJpZM4MWNO5>\n> .\n>\n"}}
{"actor":{"display_login":"zackchase"}}