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

Optimized item selection #17

Closed
wants to merge 11 commits into from
Closed

Conversation

takojunior
Copy link

  • This PR is to add the public dataset used in CPAIOR'21 paper. The data is ready to be used to run Text-based Selection for applying optimized item selection framework proposed in the paper.
  • The data includes GoodReads book reviews and MovieLens for movie recommendations. A README file is provided to introduce the data and show a usage example.

item_selection/README.md Outdated Show resolved Hide resolved
item_selection/README.md Outdated Show resolved Hide resolved
item_selection/book recommenders/goodreads_1k_data.csv Outdated Show resolved Hide resolved
item_selection/book recommenders/goodreads_1k_data.csv Outdated Show resolved Hide resolved
item_selection/book recommenders/goodreads_1k_label.csv Outdated Show resolved Hide resolved
Signed-off-by: Xin Wang <[email protected]>
Signed-off-by: Xin Wang <[email protected]>
@takojunior
Copy link
Author

Hi @skadio @bkleyn , I think I addressed all the current comments so far. Let me know if there would be further comments. If the PR looks good to go, I can merge the branch today or tomorrow.

@skadio
Copy link
Contributor

skadio commented Dec 13, 2023

@takojunior to re-iterate, we are NOT merging this branch, yes?

@takojunior
Copy link
Author

@takojunior to re-iterate, we are NOT merging this branch, yes?

Got it. Sorry, I thought we would merge the branch with this ISP data.

I can withdraw the PR if it looks good for now and keep the branch.

@skadio
Copy link
Contributor

skadio commented Dec 13, 2023

@takojunior we need to clean this up though.. let me push a quick commit

@takojunior
Copy link
Author

Hi @skadio @bkleyn , I made the following updates to the data repo per discussion:

  • Update readme by adding citation, and a few advanced usage examples, i.e. using other selection methods by changing optimization_method and cost_metric configurations
  • At the beginning mention the data is extracted from original public source and used in our research study, so ask users to refrain from distributing
  • Uploaded the datasets to huggingface under skadio/optimized_item_selection, and updated the usage example by loading the data from there
  • After the PR looks good, I think we can close it, and then we either keep this branch for further edits/reviews or delete and make changes on huggingface.

@takojunior takojunior closed this Dec 15, 2023
@takojunior takojunior deleted the optimized_item_selection branch December 15, 2023 20:21
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