{"action":"created","issue":{"url":"https://api.github.com/repos/Theano/Theano/issues/5231","repository_url":"https://api.github.com/repos/Theano/Theano","labels_url":"https://api.github.com/repos/Theano/Theano/issues/5231/labels{/name}","comments_url":"https://api.github.com/repos/Theano/Theano/issues/5231/comments","events_url":"https://api.github.com/repos/Theano/Theano/issues/5231/events","html_url":"https://github.com/Theano/Theano/pull/5231","id":189466730,"number":5231,"title":"Clarify GPU memory pre-allocation in new and old backend","user":{"login":"chinnadhurai","id":9500331,"avatar_url":"https://avatars.githubusercontent.com/u/9500331?v=3","gravatar_id":"","url":"https://api.github.com/users/chinnadhurai","html_url":"https://github.com/chinnadhurai","followers_url":"https://api.github.com/users/chinnadhurai/followers","following_url":"https://api.github.com/users/chinnadhurai/following{/other_user}","gists_url":"https://api.github.com/users/chinnadhurai/gists{/gist_id}","starred_url":"https://api.github.com/users/chinnadhurai/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chinnadhurai/subscriptions","organizations_url":"https://api.github.com/users/chinnadhurai/orgs","repos_url":"https://api.github.com/users/chinnadhurai/repos","events_url":"https://api.github.com/users/chinnadhurai/events{/privacy}","received_events_url":"https://api.github.com/users/chinnadhurai/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-11-15T18:16:00Z","updated_at":"2016-11-18T21:11:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Theano/Theano/pulls/5231","html_url":"https://github.com/Theano/Theano/pull/5231","diff_url":"https://github.com/Theano/Theano/pull/5231.diff","patch_url":"https://github.com/Theano/Theano/pull/5231.patch"},"body":"Comments in the theano documentation that config.lib.cnmem affects only the old back-end, and that config.gpuarray.preallocate is its new-backend equivalent.\r\n\r\nfix gh-5186"},"comment":{"url":"https://api.github.com/repos/Theano/Theano/issues/comments/261642914","html_url":"https://github.com/Theano/Theano/pull/5231#issuecomment-261642914","issue_url":"https://api.github.com/repos/Theano/Theano/issues/5231","id":261642914,"user":{"login":"nouiz","id":180987,"avatar_url":"https://avatars.githubusercontent.com/u/180987?v=3","gravatar_id":"","url":"https://api.github.com/users/nouiz","html_url":"https://github.com/nouiz","followers_url":"https://api.github.com/users/nouiz/followers","following_url":"https://api.github.com/users/nouiz/following{/other_user}","gists_url":"https://api.github.com/users/nouiz/gists{/gist_id}","starred_url":"https://api.github.com/users/nouiz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nouiz/subscriptions","organizations_url":"https://api.github.com/users/nouiz/orgs","repos_url":"https://api.github.com/users/nouiz/repos","events_url":"https://api.github.com/users/nouiz/events{/privacy}","received_events_url":"https://api.github.com/users/nouiz/received_events","type":"User","site_admin":false},"created_at":"2016-11-18T21:11:51Z","updated_at":"2016-11-18T21:11:51Z","body":"Mostly, it mean do a preallocation of size 0. But still cache the\nallocation. The new back-end, when the cache is enabled, when it don't have\nenough memory, it will do a new allocation to cuda and then cache it. So\nmostly, never return it to CUDA. So when that memory get freed, we keep it\nand will reuse it when we ask for new memory later.\n\nOn Fri, Nov 18, 2016 at 3:04 PM, Chinna <notifications@github.com> wrote:\n\n> *@chinnadhurai* commented on this pull request.\n> ------------------------------\n>\n> In doc/library/config.txt <https://github.com/Theano/Theano/pull/5231>:\n>\n> >\n> -    Default: 0 (but should change later)\n> +        This value allocates GPU memory ONLY when using (:ref:`gpuarray`).\n> +        For the old backend, please see ``config.lib.cnmem``\n>\n> \"0 mean enabled by don't preallocate, only cache the allocation.\" Not sure\n> I understand this. Could you please clarify?\n>\n> —\n> You are receiving this because you were mentioned.\n> Reply to this email directly, view it on GitHub\n> <https://github.com/Theano/Theano/pull/5231>, or mute the thread\n> <https://github.com/notifications/unsubscribe-auth/AALC-wGAh7ooE2-eeLbLOiOSOceHJFM_ks5q_gTEgaJpZM4Ky2Z0>\n> .\n>\n"}}
{"actor":{"display_login":"nouiz"}}