Skip to content

Conversation

ForFishes
Copy link
Member

This Op can calculate batch fc. Notice: It currently supports GPU device. This Op exists in contrib, which means that it is not shown to the public.
Part dim of batch_fc_op:
X.dim = slot_pairs_num * ins_num * in_dim
W.dim = slot_pairs_num * in_dim * out_dim
b.dim = slot_pairs_num * out_dim
output.dim = slot_pairs_num * ins_num * out_dim

Copy link
Collaborator

@gavin1332 gavin1332 left a comment

Choose a reason for hiding this comment

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

Does bias add and activation calculated separately?

Copy link
Collaborator

@gavin1332 gavin1332 left a comment

Choose a reason for hiding this comment

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

LGTM

lanxianghit
lanxianghit previously approved these changes Apr 23, 2020
def batch_fc(input, param_size, param_attr, bias_size, bias_attr, act=None):
"""
**Batch FC layer**
This Op can calculate BatcheFC.

Choose a reason for hiding this comment

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

BatcheFC是拼写错误?

Copy link
Member Author

Choose a reason for hiding this comment

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

OK, i will fix it.

def batch_fc(input, param_size, param_attr, bias_size, bias_attr, act=None):
"""
**Batch FC layer**
This Op can calculate BatcheFC.

Choose a reason for hiding this comment

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

也请解释下具体释义,目前不能直观理解op功能

Copy link
Member Author

Choose a reason for hiding this comment

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

OK, i will fix it,thanks~

@gavin1332 gavin1332 merged commit 0fb9b20 into PaddlePaddle:develop Apr 26, 2020
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.

4 participants