-
Notifications
You must be signed in to change notification settings - Fork 48
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add axis argument to softmax() (#649)
* Add axis argument to softmax() Frameworks (TensorFlow, PyTorch, ONNX) all accept an axis parameter. Most backends also support an axis, or it can be emulated with a reshape. As @fdwr wrote: So it's achievable in each backend... but it would move the pain from the caller down to where it can be handled efficiently. Fixes #466 * revert activation example to softmax * validate softmax axis against inputs rank * update TOC headers * Update index.bs Co-authored-by: Dwayne Robinson <[email protected]> * camelCase not snake_case * Remove unnecessary condition * Update index.bs Co-authored-by: Dwayne Robinson <[email protected]> * Update index.bs Co-authored-by: Dwayne Robinson <[email protected]> * Update index.bs Co-authored-by: Dwayne Robinson <[email protected]> * Sketch of validation for activations * For gru() and lstm(), calculate gate descriptor, validate activations with it * fix some copy/pasta --------- Co-authored-by: Dwayne Robinson <[email protected]>
- Loading branch information
1 parent
0df0296
commit d57e4ef
Showing
2 changed files
with
43 additions
and
21 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters