This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Quadratic operator backward actually calls forward for FComputeEx #16938

Open

kpuatamazon opened this issue Nov 28, 2019 · 0 comments

Labels

Bug

Contributor

kpuatamazon commented Nov 28, 2019 •

edited

Loading

Description

I think this line is incorrect.

https://github.com/apache/incubator-mxnet/blob/32a9baac3b7b58b133bfc4eb9426ccba75986eb3/src/operator/contrib/quadratic_op.cc#L76

It's defining the backward operator _contrib_backward_quadratic but calls the forward code.

https://github.com/apache/incubator-mxnet/blob/32a9baac3b7b58b133bfc4eb9426ccba75986eb3/src/operator/contrib/quadratic_op.cc#L70-L76

Realize this operator is mostly there for documentation and I'm trying to learn from it.

The text was updated successfully, but these errors were encountered:

kpuatamazon added the Bug label

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.