Skip to content
This repository was archived by the owner on Nov 17, 2023. It is now read-only.

[OP] Generalize SoftmaxActivation#2833

Merged
tqchen merged 4 commits intoapache:masterfrom
sbodenstein:softmax_activation_generalize
Aug 2, 2016
Merged

[OP] Generalize SoftmaxActivation#2833
tqchen merged 4 commits intoapache:masterfrom
sbodenstein:softmax_activation_generalize

Conversation

@sbodenstein
Copy link
Contributor

  1. The forward pass for SoftmaxActivation was limiting the inputs to rank-4 tensors. This restriction didn't exist in the backward pass nor for the cudnn implementation of the forward pass. This restriction is now lifted.
  2. Fixed small mistake in the docs for this symbol (used 'type' rather than 'mode' in the description).

@tqchen tqchen merged commit ea24b38 into apache:master Aug 2, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants