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

Why user_hist and user_count of val_data is not contained by val_data itself? #45

Open
pepsi2222 opened this issue Mar 21, 2023 · 1 comment

Comments

@pepsi2222
Copy link
Collaborator

the uh and uc of trn_data are added to trn_data, val_data and tst_data.
the uh and uc of val_data are added to tst_data, but why not add them to val_data itself?

The code is at the end of _build()

@XuHwang
Copy link
Collaborator

XuHwang commented Mar 28, 2023

Because the user_hist passed to tst_data contains the interactions in val_data, we pass it to test data only because when evaluation on test data, we need to use interactions in val data.

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