Skip to content

Add integration tests for aggregation options of each model #258

@tizianocitro

Description

@tizianocitro

Is your feature request related to a problem? Please describe.

Several hyperlink prediction modules expose aggregation options, but we should ensure that every advertised aggregation option actually works end-to-end for each model.

Without integration tests, a model may accept an aggregation value in its constructor but fail during forward propagation, training, validation, or testing.

Describe the solution you would like

Add parameterized integration tests for the aggregation options supported by each hyperlink prediction model.

No need to test for VilLainHlpModule with embedding_mode="hyperedge", as the aggregation option is ignored there.

Describe alternatives you've considered

NA.

Additional context

Acceptance criteria:

  • Tests are parameterized by model and aggregation option.
  • Every aggregation option is covered for each model.
  • Tests fail with a clear model/aggregation combination when something breaks.
  • Unsupported aggregation values are not included in the success matrix.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions