cupy icon indicating copy to clipboard operation
cupy copied to clipboard

[RFC] Drop support for cuDNN in CuPy v14

Open kmaehashi opened this issue 2 years ago • 7 comments

As per the discussion below we are considering to drop support for cuDNN in CuPy. Feel free to leave a comment if you rely on cuDNN features in CuPy.

kmaehashi avatar Feb 28 '24 06:02 kmaehashi

@kmaehashi I wanted to let you know that we have the new cuDNN Frontend that may make integration and fusions easier. Let me know if you have any questions

mnicely avatar Mar 01 '24 16:03 mnicely

Thanks for the info, @mnicely! It's great to see the stable release of an official high-level library with Python binding!

Actually, CuPy's cuDNN wrappers haven't been updated for years - it has been maintained to support new cuDNN releases but not adopting new features. They were originally added for use by Chainer and I think most users today yare not relying on the CuPy cuDNN features. It's indicated by that there are no bug reports nor feature requests recently.

Now that cuDNN wrappers are officially provided, I think it would make sense to deprecate cuDNN support in CuPy.

cc/ @leofang

kmaehashi avatar Mar 12 '24 02:03 kmaehashi

Now that cuDNN wrappers are officially provided, I think it would make sense to deprecate cuDNN support in CuPy.

cc: @acc-mu3n for vis (for context in the private Gitter discussion)

leofang avatar Mar 21 '24 16:03 leofang

Hi @honnibal you mentioned in the past that spaCy uses cuDNN through CuPy. Is it still the case? We wanna estimate the impact of dropping support of cuDNN in CuPy, but oddly we can't find any trace of cuDNN in the spaCy codebase (either its addition or removal!), and we are quite puzzled. Could you help us understand better?

cc: @jakirkham for vis

leofang avatar Mar 22 '24 01:03 leofang

@kmaehashi shall we repurpose this issue and make it an RFC to drop cuDNN support by CuPy v14?

leofang avatar Aug 28 '24 04:08 leofang

Updated the issue title & body.

kmaehashi avatar Sep 17 '24 13:09 kmaehashi