Francisco Massa
Francisco Massa
@CodeRect it works in some (most?) cases, but there are some situations where I think it messes up with some functionalities, such as `type()`. I didn't have the time to...
@soumith I'll have a look at that tomorrow. I don't have access anymore to the models that were failing, but I'll try to reproduce the failure case I had.
@soumith sorry, I got stuck with my PhD writing. but I'm submitting a final version to the reviewers this week so I will have time to look into fixing this...
@suryabhupa hey, sorry about that. I got busy with other things and I didn't have the time to look into that yet. Not sure when I'll find some time though,...
Nice! I think that it could break saved models that haven't the `iSize` field (i.e., when you save a model before ever forwarding it). Wouldn't it be better to simply...
@soumith that happened at least once in the past with me https://github.com/torch/nn/pull/712#issuecomment-196573630 , but I'd say it's a fair enough change :)
You can have nn modules running on the GPU, and you can mix nn and cudnn modules, but they all should be on gpu in this case. You need to...
Could you write a small working example that illustrates the issue?
@etrulls this is being tracked in here https://github.com/soumith/cudnn.torch/issues/50 .
Quick note: there is no need to implement `LinearNB`, as the no-bias functionality was integrated in `nn` already https://github.com/torch/nn/pull/583