[GitHub] [incubator-mxnet] apeforest edited a comment on issue #15120: [bug] fix higher grad log

2019-06-06 Thread GitBox
apeforest edited a comment on issue #15120: [bug] fix higher grad log URL: https://github.com/apache/incubator-mxnet/pull/15120#issuecomment-499608736 @kshitij12345 I think it's because of the design of Python backward API in MXNet. When you specify `variables=x`, MXNet will only compute

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #15120: [bug] fix higher grad log

2019-06-06 Thread GitBox
apeforest edited a comment on issue #15120: [bug] fix higher grad log URL: https://github.com/apache/incubator-mxnet/pull/15120#issuecomment-499608736 @kshitij12345 I think it's because of the design of Python backward API in MXNet. When you specify `variables=x`, MXNet will only compute

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #15120: [bug] fix higher grad log

2019-06-06 Thread GitBox
apeforest edited a comment on issue #15120: [bug] fix higher grad log URL: https://github.com/apache/incubator-mxnet/pull/15120#issuecomment-499608736 @kshitij12345 I think it's because of the design of Python backward API in MXNet. When you specify `variables=x`, MXNet will only compute

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #15120: [bug] fix higher grad log

2019-06-06 Thread GitBox
apeforest edited a comment on issue #15120: [bug] fix higher grad log URL: https://github.com/apache/incubator-mxnet/pull/15120#issuecomment-499599418 > ```python > x_grad = x_grad_mid * y_grad # Note this part. > ``` I don't think this part is correct. Note: your

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #15120: [bug] fix higher grad log

2019-06-05 Thread GitBox
apeforest edited a comment on issue #15120: [bug] fix higher grad log URL: https://github.com/apache/incubator-mxnet/pull/15120#issuecomment-499170409 @kshitij12345 I have some question about the equation `expected_head_grad = (grad_op(x) * head_grad_grads).asnumpy()` in your test.

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #15120: [bug] fix higher grad log

2019-06-05 Thread GitBox
apeforest edited a comment on issue #15120: [bug] fix higher grad log URL: https://github.com/apache/incubator-mxnet/pull/15120#issuecomment-499192329 I did some more probing. I think the reason that head_grads.grad being all zeros is the variable head_grads was not specified during the

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #15120: [bug] fix higher grad log

2019-06-05 Thread GitBox
apeforest edited a comment on issue #15120: [bug] fix higher grad log URL: https://github.com/apache/incubator-mxnet/pull/15120#issuecomment-499192329 I did some more probing. I think the reason that head_grads.grad being all zeros is the variable head_grads was not specified during the

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #15120: [bug] fix higher grad log

2019-06-05 Thread GitBox
apeforest edited a comment on issue #15120: [bug] fix higher grad log URL: https://github.com/apache/incubator-mxnet/pull/15120#issuecomment-499192329 I did some more probing. I think the reason that head_grads.grad being all zeros is the variable head_grads was not specified during the

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #15120: [bug] fix higher grad log

2019-06-05 Thread GitBox
apeforest edited a comment on issue #15120: [bug] fix higher grad log URL: https://github.com/apache/incubator-mxnet/pull/15120#issuecomment-499170409 @kshitij12345 I have some question about the equation `expected_head_grad = (grad_op(x) * head_grad_grads).asnumpy()` in your test.

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #15120: [bug] fix higher grad log

2019-06-05 Thread GitBox
apeforest edited a comment on issue #15120: [bug] fix higher grad log URL: https://github.com/apache/incubator-mxnet/pull/15120#issuecomment-499192329 I did some more probing. I think the reason that head_grads.grad being all zeros is the variable head_grads was not specified during the

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #15120: [bug] fix higher grad log

2019-06-05 Thread GitBox
apeforest edited a comment on issue #15120: [bug] fix higher grad log URL: https://github.com/apache/incubator-mxnet/pull/15120#issuecomment-499192329 I did some more probing. I think the reason that head_grads.grad being all zeros is the variable head_grads was not specified during the

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #15120: [bug] fix higher grad log

2019-06-05 Thread GitBox
apeforest edited a comment on issue #15120: [bug] fix higher grad log URL: https://github.com/apache/incubator-mxnet/pull/15120#issuecomment-499170409 @kshitij12345 I have some question about the equation `expected_head_grad = (grad_op(x) * head_grad_grads).asnumpy()` in your test.

[GitHub] [incubator-mxnet] apeforest edited a comment on issue #15120: [bug] fix higher grad log

2019-06-05 Thread GitBox
apeforest edited a comment on issue #15120: [bug] fix higher grad log URL: https://github.com/apache/incubator-mxnet/pull/15120#issuecomment-499170409 @kshitij12345 I have some question about the equation `expected_head_grad = (grad_op(x) * head_grad_grads).asnumpy()` in your test.