Skip to content
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

Torch7 RNN Benchmarks #14

Open
dasguptar opened this issue May 2, 2017 · 1 comment
Open

Torch7 RNN Benchmarks #14

dasguptar opened this issue May 2, 2017 · 1 comment

Comments

@dasguptar
Copy link

Hi,

Really appreciate the work put in by you guys to compare all these libraries across so many different models and configurations.
I had a minor nitpick. For Torch7, it seems you are using https://github.com/Element-Research/rnn.
However, this library, as far as I know, does not leverage CuDNN bindings or specialised CUDA kernels for RNNs, something which is par for the course for other libraries such as TF, CNTK, etc.
For Torch7, you should be using either:

These libraries have specialised CUDA kernels for RNNs, and should bring the RNN benchmarks for Torch7 up to a comparable position compared to other libraries being benchmarked.

@shyhuai
Copy link
Collaborator

shyhuai commented May 3, 2017

Many thanks for your comments. We will include the cuDNN version of RNNs for Torch7 in the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants