Skip to content

Add support for cuDNN v6#573

Merged
weiliu89 merged 1 commit intoweiliu89:ssdfrom
thundo:ssd
May 9, 2017
Merged

Add support for cuDNN v6#573
weiliu89 merged 1 commit intoweiliu89:ssdfrom
thundo:ssd

Conversation

@thundo
Copy link
Copy Markdown

@thundo thundo commented May 6, 2017

Cherrypicked BVLC@0096fe3. It should resolve #534 and #572.

Thank you

Support for cuDNN v4 and v5 is preserved.
@wk910930
Copy link
Copy Markdown

wk910930 commented May 8, 2017

Dear @thundo , I merged and tested your code with cuDNN v6. However, the training speed looks similar, not faster as expected. What is your observation?

@thundo
Copy link
Copy Markdown
Author

thundo commented May 8, 2017

Point is that, without this PR, I cannot build with cuDNN v6 installed on my system. This range of commits should copy a workaround (present in BVLC/caffe) for supporting v5 and v6 both. See #572 for build log.

@wk910930
Copy link
Copy Markdown

wk910930 commented May 8, 2017

Got it. Thanks!

Copy link
Copy Markdown

@wliu-nuro wliu-nuro left a comment

Choose a reason for hiding this comment

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

Thanks for adding it!

@weiliu89 weiliu89 merged commit 9d6e815 into weiliu89:ssd May 9, 2017
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.

4 participants