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

Is sentiment required? #24

Open
yangb0o opened this issue Oct 12, 2024 · 4 comments
Open

Is sentiment required? #24

yangb0o opened this issue Oct 12, 2024 · 4 comments

Comments

@yangb0o
Copy link

yangb0o commented Oct 12, 2024

Hi,

I notice that in the model you added sentiment for training, may I ask how to remove it?

@andreeaiana
Copy link
Owner

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.

@yangb0o
Copy link
Author

yangb0o commented Oct 17, 2024

Hi,
I can remove some attributes in config, for example:

# 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?

@andreeaiana
Copy link
Owner

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.

@yangb0o
Copy link
Author

yangb0o commented Oct 23, 2024

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.

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