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

Support cuDNN 9.0 #8215

Open
6 tasks
kmaehashi opened this issue Feb 28, 2024 · 4 comments
Open
6 tasks

Support cuDNN 9.0 #8215

kmaehashi opened this issue Feb 28, 2024 · 4 comments
Assignees
Labels
cat:enhancement Improvements to existing features prio:medium

Comments

@kmaehashi
Copy link
Member

Tasks

  • Read Release Notes (cuTENSOR / cuSPARSELt / cuDNN / NCCL).
  • Fix code and CI to support the new vesrion.
    • FlexCI: Update .pfnci/schema.yaml and .pfnci/matirx.yaml.
    • Library Installer: Update cupyx/tools/install_library.py.
  • Backport the above PR.
  • Fix documentation.
  • Backport the above PR.
  • Implement new features using the version, if any.
@kmaehashi kmaehashi self-assigned this Feb 29, 2024
@kmaehashi kmaehashi added cat:enhancement Improvements to existing features prio:medium labels Feb 29, 2024
@mnicely
Copy link
Contributor

mnicely commented Mar 1, 2024

@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

@kmaehashi
Copy link
Member Author

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

@leofang
Copy link
Member

leofang commented Mar 21, 2024

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
Copy link
Member

leofang commented Mar 22, 2024

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat:enhancement Improvements to existing features prio:medium
Projects
None yet
Development

No branches or pull requests

3 participants