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

[feature] leave-k-out split mode #2121

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

mkhe93
Copy link
Contributor

@mkhe93 mkhe93 commented Nov 30, 2024

  • Introduced a new leave-k-out split mode, as already available in Elliot (see: Elliot paper).
  • the leave-k-out mode allows for excluding a specified number of elements (denoted as k) from the dataset, based on the chosen mode
  • updated the documentation to reflect this new split mode and provide usage instructions

Example in eval_args in example.yaml for k=10:

split: {'LK': ['test_only', 10]}

@mkhe93
Copy link
Contributor Author

mkhe93 commented Nov 30, 2024

Created this mode during my researches and Recbole improved my workflow a lot.
I'm happy for feedback and improvements!

@TayTroye
Copy link
Collaborator

@mkhe93
Hello, thank you for your attention. The commonly used partitioning method in research study is leave-one-out, and you are welcome to use the leave-k-out method as needed. At this time, we do not have plans to integrate it into RecBole.

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.

2 participants