{"comment":{"created_at":"2011-10-21T15:02:20Z","body":"Thanks Razvan, just moving it out of the global import path is a good\nenough for now I think. Put a comment next to the internal import\ndocumenting why it isn't a global import, and that'd be fine by me.\n\n- James\n\nOn Fri, Oct 21, 2011 at 10:45 AM, Razvan Pascanu\n<reply@reply.github.com>\nwrote:\n> Hmm .. sorry for that. I somehow missed that import ( I import cuda also in\n> scan_op and scan_utils, but there only inside the functions that need it).\n> I've moved the import cuda inside the scan function. Hopefully this will\n> make things better for those who don't use scna.\n>\n> I'll work on removing it completely, but I do not have a lot of time to\n> dedicate to Theano\n> unfortunately...\n>\n> Razvan\n>\n> On Fri, Oct 21, 2011 at 10:40 AM, James Bergstra <\n> reply@reply.github.com>wrote:\n>\n>> This seems straightforward, no?\n>>\n>> 1) https://github.com/Theano/Theano/blob/master/theano/__init__.py#L80\n>>\n>> 2)\n>> https://github.com/Theano/Theano/blob/master/theano/scan_module/scan.py#L56\n>>\n>>\n>> On Fri, Oct 21, 2011 at 10:35 AM, Razvan Pascanu\n>> <reply@reply.github.com>\n>> wrote:\n>> > I've looked through the code. cuda is imported only inside two functions\n>> > (exactly where I need it) when you use scan. So is it that the code that\n>> > generates the error actually uses scan, or this error happens even if you\n>> do\n>> > not use scan?\n>> >\n>> > Razvan\n>> >\n>> >\n>> > On Fri, Oct 21, 2011 at 8:39 AM, Razvan Pascanu <r.pascanu@gmail.com>\n>> wrote:\n>> >\n>> >> Yes I think so. This was introduced long long time ago. I've tried at\n>> one\n>> >> point to fix it (before we moved to git) but it turned out not to be a\n>> >> trivial task. I'll give it another go,\n>> >> but know that this will not be done in a matter of minutes :(. Maybe I\n>> can\n>> >> disable scan working on GPU for now removing the import (that should be\n>> >> quick) and work on\n>> >> repairing the damage afterwards.\n>> >>\n>> >> Razvan\n>> >>\n>> >>\n>> >> On Thu, Oct 20, 2011 at 8:53 PM, David Warde-Farley <\n>> >> reply@reply.github.com>wrote:\n>> >>\n>> >>> Is this in theano stable as well?\n>> >>>\n>> >>> --\n>> >>> Reply to this email directly or view it on GitHub:\n>> >>> https://github.com/Theano/Theano/issues/134#issuecomment-2476578\n>> >>>\n>> >>\n>> >>\n>> >\n>> > --\n>> > Reply to this email directly or view it on GitHub:\n>> > https://github.com/Theano/Theano/issues/134#issuecomment-2481965\n>> >\n>>\n>> --\n>> Reply to this email directly or view it on GitHub:\n>> https://github.com/Theano/Theano/issues/134#issuecomment-2482018\n>>\n>\n> --\n> Reply to this email directly or view it on GitHub:\n> https://github.com/Theano/Theano/issues/134#issuecomment-2482079\n>","updated_at":"2011-10-21T15:02:20Z","url":"https://api.github.com/repos/Theano/Theano/issues/comments/2482292","id":2482292,"user":{"gravatar_id":"97d987b98f02e8e5c1d8a6c5350b06c1","avatar_url":"https://secure.gravatar.com/avatar/97d987b98f02e8e5c1d8a6c5350b06c1?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","url":"https://api.github.com/users/jaberg","id":171276,"login":"jaberg"}},"action":"created","issue":{"number":134,"created_at":"2011-10-20T20:58:31Z","pull_request":{"diff_url":null,"patch_url":null,"html_url":null},"body":"Can we remove this import?  I thought this had been discussed / done before.  Reasons to remove it are\r\n\r\n1) non-sandbox code should not depend on sandbox code\r\n\r\n2) scan should not depend on the GPU being available\r\n\r\nIn practice, it the combination of Theano + GPU is still often pain to install / configure. One would like to be able to just skip it.\r\n\r\nIt makes Theano look bad when someone sets device=cpu and force_device=1 to avoid having to do that configuration, only to be greeted by 3500 lines of C code from a failed call to nvcc.","comments":6,"title":"scan module imports sandbox.cuda - hilarities ensue on badly configured OS-X","updated_at":"2011-10-21T15:02:20Z","url":"https://api.github.com/repos/Theano/Theano/issues/134","id":2009343,"assignee":{"gravatar_id":"57c594d3d0f97d3010b15b6a0806451c","avatar_url":"https://secure.gravatar.com/avatar/57c594d3d0f97d3010b15b6a0806451c?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","url":"https://api.github.com/users/pascanur","id":178095,"login":"pascanur"},"milestone":null,"closed_at":null,"user":{"gravatar_id":"97d987b98f02e8e5c1d8a6c5350b06c1","avatar_url":"https://secure.gravatar.com/avatar/97d987b98f02e8e5c1d8a6c5350b06c1?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","url":"https://api.github.com/users/jaberg","id":171276,"login":"jaberg"},"html_url":"https://github.com/Theano/Theano/issues/134","labels":[],"state":"open"},"legacy":{"issue_id":2009343,"comment_id":2482292}}