Skip to content
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

seems no usage of dropout #15

Open
hzwudi2014 opened this issue May 18, 2018 · 3 comments
Open

seems no usage of dropout #15

hzwudi2014 opened this issue May 18, 2018 · 3 comments

Comments

@hzwudi2014
Copy link

hello, I don't find dropout param in the code
but the paper says dropout can lift the performance
is there any implementation of dropout in this project?

@Leavingseason
Copy link
Owner

Humm.... Yes I didn't use dropout logic in this version since we have started a new project named DeepRec, which includes deepFM and contains dropout logic. This project is expected to be released in about one or two months. If you really want to run a deepFM model with dropout config, tell me and I will add the logic in about 2 days.

@hzwudi2014
Copy link
Author

Thank u for replying
I would really appreciate it if u can add the dropout logic in the code.
Looking forward to ur new project

@Leavingseason
Copy link
Owner

I have added the dropout code for the deepFM script.

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

No branches or pull requests

2 participants