-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add text classification example #684
Conversation
5b8062f
to
349006d
Compare
Hi Greg. This is awesome. Read the paper earlier as well and thought it was awesome. Thanks man, this is very cool!
And the verbose output of torch is
Weird. I might try recreating the dataset to make absolutely sure i made it correctly. However, did seem to pass the Validation epoch (7.12% accuracy epoch 0) which tells me that the model and the data are probably legit. update
update update 2 |
Hi @TimZaman thanks for the feedback! I used |
hi @TimZaman are you still having an issue with this example? |
Hi @gheinrich, i did not use mean substraction as i was supposed to. |
I tested the dataset creation and import. When I tried to test the model, I got this error:
Maybe it's because I don't have this commit: soumith/cudnn.torch#152? I had to start with a non-standard fork (16.04 issues), so I'm having trouble updating just the |
Oh so it's the same error @TimZaman got. You could update just $ cd .../torch/extra/cudnn
$ git fetch origin && git checkout origin/master
$ luarocks make cudnn-scm-1.rockspec |
I don't know of other advantages... |
Yeah so the cudnn stuff is not DIGITS related. @gheinrich I would recommend just leaving out the cudnn, switch to nn, albeit 17x slower (worked fine with me, but maybe make a note about cudnn being faster but you'd have to update) and just merge. |
349006d
to
5861685
Compare
Add text classification example
No description provided.