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 link to CuPy upgrade guide #4188
Conversation
5f0a104
to
d132d6c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The links seem wrong to me. Could you let me check my understanding?
docs/source/upgrade.rst
Outdated
------- | ||
|
||
Chainer v4 requires CuPy v4 if you need GPU support. | ||
Please see the `Upgrade Guide for CuPy v4 <https://docs.chainer.org/en/latest/upgrade.html#cupy-v4>`_ for details. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The link seems points to Chainer's document, not CuPy. Is that correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, that was wrong. Fixed.
docs/source/upgrade.rst
Outdated
------- | ||
|
||
Chainer v3 requires CuPy v2 if you need GPU support. | ||
Please see the `Upgrade Guide for CuPy v2 <https://docs.chainer.org/en/latest/upgrade.html#cupy-v2>`_ for details. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
jenkins, test this please. |
Thank you for update. I'm thinking to merge this after cupy/cupy#884. |
Jenkins, test this please. |
LGTM |
add link to CuPy upgrade guide
Add link to CuPy upgrade guide from Chaier upgrade guide.
Depends on cupy/cupy#884.(merged)