ptrendx opened a new pull request #17049: Fix operators lying about their number of inputs URL: https://github.com/apache/incubator-mxnet/pull/17049 ## Description ## This PR fixes a number of backward operators that advertise the wrong number of inputs (either by having a wrong number passed to `set_num_inputs` or leaving it empty which defaults to 1 input). It also introduces a check in `MakeNode` to catch this error. Having consistent number of inputs and outputs is important because tools like `Monitor` rely on them (and currently some of the networks are impossible to inspect during backward because of this) and they are needed for more advanced manipulation of the computational graph. ## Checklist ## ### Essentials ### Please feel free to remove inapplicable items for your PR. - [ ] Changes are complete (i.e. I finished coding on this PR)
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
