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

Add Learn to Rank user Preferences based on Phrase-level sentiment analysis across Multiple categories (LRPPM) model #527

Merged
merged 1 commit into from
Oct 15, 2023

Conversation

lthoang
Copy link
Member

@lthoang lthoang commented Sep 14, 2023

…alysis across Multiple categories (LRPPM) model

Description

Related Issues

Checklist:

  • I have added tests.
  • I have updated the documentation accordingly.
  • I have updated README.md (if you are adding a new model).
  • I have updated examples/README.md (if you are adding a new example).
  • I have updated datasets/README.md (if you are adding a new dataset).

@lthoang lthoang requested review from tqtg and saghiles September 14, 2023 09:19
examples/efm_example.py Outdated Show resolved Hide resolved
@tqtg
Copy link
Member

tqtg commented Sep 24, 2023

Overall structure looks good to me, though I'm not able to check the logic of algorithm in detail. Please feel free to merge as if you're comfortable with the implementation. Thanks.

@lthoang lthoang force-pushed the lrppm branch 3 times, most recently from 46685c1 to 38694f4 Compare September 25, 2023 08:06
…alysis across Multiple categories (LRPPM) model
@tqtg
Copy link
Member

tqtg commented Oct 13, 2023

@lthoang shall we merge it? Or do we expect further changes to the model?

@lthoang lthoang merged commit c484988 into PreferredAI:master Oct 15, 2023
12 checks passed
@lthoang lthoang deleted the lrppm branch October 15, 2023 08:57
darrylong pushed a commit to darrylong/cornac that referenced this pull request Oct 23, 2023
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