Skip to content

Unify Activation functions and simplify register code#4485

Merged
reyoung merged 2 commits intoPaddlePaddle:developfrom
reyoung:feature/BetterActivationKern
Sep 29, 2017
Merged

Unify Activation functions and simplify register code#4485
reyoung merged 2 commits intoPaddlePaddle:developfrom
reyoung:feature/BetterActivationKern

Conversation

@reyoung
Copy link
Collaborator

@reyoung reyoung commented Sep 28, 2017

No description provided.

@reyoung reyoung requested a review from QiJune September 28, 2017 23:27
@reyoung reyoung force-pushed the feature/BetterActivationKern branch from f794396 to 337b7eb Compare September 28, 2017 23:56
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why here attr is always float? Attr has its own data type, and will be static_cast to activation functor data type

@reyoung reyoung force-pushed the feature/BetterActivationKern branch from eec6ed5 to a8c6ce9 Compare September 29, 2017 03:16
Copy link
Member

@QiJune QiJune left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@reyoung reyoung merged commit ae4b7fd into PaddlePaddle:develop Sep 29, 2017
@reyoung reyoung deleted the feature/BetterActivationKern branch October 2, 2017 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants