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

add cuDNN v5 support, drop cuDNN v3 support #8

Merged
merged 3 commits into from
Oct 25, 2016

Conversation

ningzhou
Copy link
Contributor

The change was purely based on BVLC/caffe#4159

Copy link
Collaborator

@Cysu Cysu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please consider switch back to the original lib64 path? As other linux distributions (ubuntu 14.04, arch linux) do not have lib/x86_64-linux-gpu under cuda.

@Cysu Cysu merged commit d925485 into ShuangLI59:master Oct 25, 2016
@Cysu
Copy link
Collaborator

Cysu commented Oct 25, 2016

@ningzhou I have merged the PR. Thank you very much!

@ningzhou
Copy link
Contributor Author

You are very welcome. In fact, thank you very much for open sourcing the
excellent work.

Ning

On Mon, Oct 24, 2016 at 11:19 PM, Tong Xiao [email protected]
wrote:

@ningzhou https://github.com/ningzhou I have merged the PR. Thank you
very much!


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#8 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABNQXuPRseEJkAppTdHJBNimWOv1qsbwks5q3Z9ogaJpZM4KfjGE
.

@ningzhou
Copy link
Contributor Author

sure. I will create a new pull request.

On Mon, Oct 24, 2016 at 10:05 PM, Tong Xiao [email protected]
wrote:

@Cysu requested changes on this pull request.

Could you please consider switch back to the original lib64 path? As other
linux distributions (ubuntu 14.04, arch linux) do not have
lib/x86_64-linux-gpu under cuda.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#8 (review),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABNQXjTnD9DwRtBbcHLKxuF3JWGK1L34ks5q3Y4CgaJpZM4KfjGE
.

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

Successfully merging this pull request may close these issues.

2 participants