-
Notifications
You must be signed in to change notification settings - Fork 8
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
Is sentiment required? #24
Comments
Hi, you can specify which dataset attributes you want to use in the dataset config. If you don't want to use the sentiment information, you can comment it out. |
Hi, # configs/data/mind_rec.yaml
# Data preprocessing
dataset_attributes:
- "title"
- "abstract"
- "category"
- "subcategory"
- "title_entities"
- "abstract_entities"
- "category_class"
- "subcategory_class"
# - "sentiment_class"
# - "sentiment_score" But it seems that it only affect data preprocessing, the model (such as NRMS) still relies on sentiment information for training. I can only manually remove the module about sentiment from the model's code, can this be done by way of configuration? |
Currently, all models use the sentiment for the evaluating the diversity of the recommendations w.r.t. sentiment, not for training the model itself. I'm planning to refactor the code to make it more modular and link more settings, like the one you mentioned, directly to the configuration of the model itself. However, I'll only have capacity for this at the end of the year. For now, you'd have to manually comment out those few lines that refer to sentiments from the model's evaluation code. |
Thanks for your reply. Looking forward to the better modular. |
Hi,
I notice that in the model you added sentiment for training, may I ask how to remove it?
The text was updated successfully, but these errors were encountered: