-
Notifications
You must be signed in to change notification settings - Fork 2.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement DeepFM for CTR prediction #485
Conversation
…ification fix data reader error of text classification.
deep_fm/README.md
Outdated
second_order = paddle.layer.factorization_machine(input=input, factor_size=factor_size) | ||
fm = paddle.layer.addto(input=[first_order, second_order], | ||
act=paddle.activation.Linear(), | ||
ias_attr=False) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ias_attr --> bias_attr
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已改
deep_fm/README.md
Outdated
``` | ||
|
||
## Evaluate | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
删掉 76 行。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已删
deep_fm/README.md
Outdated
@@ -0,0 +1,90 @@ | |||
# Deep Factorization Machines (DeepFM) for Click-Through Rate prediction |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deep Factorization Machine for Click-Through Rate prediction
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已改.
deep_fm/README.md
Outdated
A Factorization-Machine based Neural Network for CTR Prediction. | ||
Proceedings of the Twenty-Sixth International Joint Conference on | ||
Artificial Intelligence (IJCAI-17), 2017 | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
line 7 ~ 10 行之前的空格删掉。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已改
deep_fm/README.md
Outdated
Artificial Intelligence (IJCAI-17), 2017 | ||
``` | ||
|
||
The DeepFm combines factorization machines and deep neural networks to model |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
factorization machine
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已改
deep_fm/infer.py
Outdated
'--model_gz_path', | ||
type=str, | ||
required=True, | ||
help="path of model parameters gz file") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
参数的解释是一句完整的话,都变成大写字母开头,句号结尾吧。
Path 前面加上 the,下同。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已改
input=[first_order, second_order], | ||
act=paddle.activation.Linear(), | ||
bias_attr=False) | ||
return out |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
line 7 ~ 19 可以作为一个helper 加入在Paddle repo下。这个等Paddle 下面的 PR merge 之后再加吧。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
好的
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merge 之后也更新一下主页的README吧。
No description provided.