Skip to content

Make it possible to specify a FeatureModality directly #374

Merged
saghiles merged 3 commits intoPreferredAI:masterfrom
saghiles:feature_modality
Dec 15, 2020
Merged

Make it possible to specify a FeatureModality directly #374
saghiles merged 3 commits intoPreferredAI:masterfrom
saghiles:feature_modality

Conversation

@saghiles
Copy link
Copy Markdown
Member

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).

@saghiles saghiles requested a review from qtuantruong December 14, 2020 07:58
Copy link
Copy Markdown
Member

@qtuantruong qtuantruong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks fine. Could you please add a few test cases to cover the new changes?

@saghiles saghiles merged commit 09ca03f into PreferredAI:master Dec 15, 2020
@saghiles saghiles deleted the feature_modality branch December 28, 2020 04:19
qtuantruong pushed a commit to qtuantruong/cornac that referenced this pull request Oct 25, 2025
qtuantruong pushed a commit to qtuantruong/cornac that referenced this pull request Oct 25, 2025
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