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

GDCN implementation #73

Merged
merged 7 commits into from
Nov 2, 2023
Merged

GDCN implementation #73

merged 7 commits into from
Nov 2, 2023

Conversation

sdilbaz
Copy link
Contributor

@sdilbaz sdilbaz commented Oct 30, 2023

Implementation of Towards Deeper, Lighter and Interpretable Cross Network for CTR Prediction https://dl.acm.org/doi/pdf/10.1145/3583780.3615089

xpai and others added 2 commits October 29, 2023 18:25
Implementation of Towards Deeper, Lighter and Interpretable Cross Network for CTR Prediction
https://dl.acm.org/doi/pdf/10.1145/3583780.3615089
@zhujiem
Copy link
Contributor

zhujiem commented Oct 31, 2023

Thanks for your pr! Some miniors to fix:

  1. Could you pls use your own copyright for GDCN.py?
  2. Please add your model info to the table in FuxiCTR/README.md

Added GDCN model
add copyright for gdcn
@sdilbaz sdilbaz closed this Oct 31, 2023
@sdilbaz sdilbaz reopened this Oct 31, 2023
@sdilbaz
Copy link
Contributor Author

sdilbaz commented Oct 31, 2023

The fixes are done 👍

README.md Outdated
| 49 | KDD'18 | [PLE](./model_zoo/multitask/PLE) | [Progressive Layered Extraction (PLE): A Novel Multi-Task Learning (MTL) Model for Personalized Recommendations](https://dl.acm.org/doi/10.1145/3383313.3412236) :triangular_flag_on_post:**Tencent** | | `torch` |
| 48 | MachineLearn'97 | [SharedBottom](./model_zoo/multitask/SharedBottom) | [Multitask Learning](https://link.springer.com/article/10.1023/A:1007379606734) | | `torch` |
| 49 | KDD'18 | [MMoE](./model_zoo/multitask/MMOE) | [Modeling Task Relationships in Multi-task Learning with Multi-Gate Mixture-of-Experts](https://dl.acm.org/doi/pdf/10.1145/3219819.3220007) :triangular_flag_on_post:**Google** | | `torch` |
| 50 | KDD'18 | [PLE](./model_zoo/multitask/PLE) | [Progressive Layered Extraction (PLE): A Novel Multi-Task Learning (MTL) Model for Personalized Recommendations](https://dl.acm.org/doi/10.1145/3383313.3412236) :triangular_flag_on_post:**Tencent** | | `torch` |
|<tr><th colspan=6 align="center">:open_file_folder: **Multi-Domain Modeling**</th></tr>|
| 50 | Arxiv'23 | PEPNet | [PEPNet: Parameter and Embedding Personalized Network for Infusing with Personalized Prior Information](https://arxiv.org/abs/2302.01115) :triangular_flag_on_post:**KuaiShou** | | `torch` |
Copy link
Contributor

Choose a reason for hiding this comment

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

--> 51 here

Copy link
Contributor

@zhujiem zhujiem left a comment

Choose a reason for hiding this comment

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

2 minors to fix, thx!

Copy link
Contributor

Choose a reason for hiding this comment

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

Please modify the file header here to your copyright.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@xpai xpai force-pushed the main branch 2 times, most recently from 861bf8e to ba21c2c Compare November 1, 2023 06:34
@xpai xpai merged commit 7495b4e into reczoo:main Nov 2, 2023
@xpai
Copy link
Member

xpai commented Nov 2, 2023

Get merged. Thanks for your contribution!

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.

3 participants