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 higher-order derivative support of Chainer to the comparison table #3477
Conversation
How about writing a note describing why this is "partial"? |
I added comments. |
LGTM. |
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.
I added one comment.
docs/source/comparison.rst
Outdated
@@ -37,8 +37,9 @@ This table compares Chainer with other actively developed deep learning framewor | |||
.. [1] Define-by-run is in development as of June 2017 and tracked in `dmlc/mxnet#5705 <https://github.com/dmlc/mxnet/pull/5705>`_. It is also possible using the much slower MinPy extension. | |||
.. [2] Symbolic autograd is in development as of June 2017 and tracked in `deeplearning4j/nd4j#1750 <https://github.com/deeplearning4j/nd4j/pull/1750>`_. | |||
.. [3] Symbolic autograd is available only with ngraph backend (experimental). | |||
.. [4] Nervana provides kernels that are meant to compete with cuDNN. | |||
.. [5] Multiprocessing provides a significant performance improvement only for frameworks that use Python at runtime. | |||
.. [4] Some functions do not support higher-order differentiation. See `chainer/chainer#2970 <https://github.com/chainer/chainer/pull/2970>`_. |
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.
Could you replace this with #4449 ?
Thank you for your review. I applied your suggestion. |
Jenkins, test this please |
LGTM |
Add higher-order derivative support of Chainer to the comparison table
This PR updates the comparison table to add partial support of higher-order derivative to the Chainer slot.