Skip to content

Conversation

chengduoZH
Copy link
Contributor

@chengduoZH chengduoZH commented Mar 28, 2018

fix #9446

@chengduoZH chengduoZH force-pushed the feature/add_cos branch 3 times, most recently from 017a5bb to a16b150 Compare March 28, 2018 12:51
@chengduoZH chengduoZH force-pushed the feature/add_cos branch 2 times, most recently from bc0bf20 to da73594 Compare March 28, 2018 15:17
@chengduoZH chengduoZH changed the title [WIP] Add cos Add cos and sin Mar 29, 2018
@chengduoZH chengduoZH requested a review from qingqing01 March 29, 2018 01:40
Copy link
Contributor

@qingqing01 qingqing01 left a comment

Choose a reason for hiding this comment

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

LGTM.

@chengduoZH chengduoZH merged commit 24100e1 into PaddlePaddle:develop Mar 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing Cosine operator

2 participants