Skip to content

Allow all recommenders to receive ICM to simplify API #14

@tomaz-suller

Description

@tomaz-suller

A pattern often employed in e.g. scikit-learn is allowing models to receive parameters which don't affect model outputs to maintain a consistent API. For an example, see the groups parameter in sklearn.model_selection.KFold.split:

groups: object
       Always ignored, exists for compatibility.

Doing something similar with recommender models that do not use the ICM could make interacting with multiple recommenders easier, since we would not need to check whether the recommender takes the ICM and then pass it, but instead pass it every time and leave the implementation to ignore it.

Scripts such as HyperparameterTuning/run_hyperparameter_search.py could then benefit and not require multiple implementations of the same logic only because some models accept the ICM while others don't.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions