search

dennlinger / summaries

A toolkit for summarization analysis and aspect-based summarizers
MIT License
11 stars 0 forks source link

Introduce base class for `Aligners` #18

Closed dennlinger closed 2 years ago

dennlinger commented 2 years ago

Given that the behavior between different Aligner methods is relatively similar, it would make sense to also inherit from a base class, such that we can easily guarantee shared parameter choices etc.

However, at the current state, this is not so much a priority, as they are more or less exclusively used for testing at the moment.

dennlinger commented 2 years ago

Resolved with e00cc53. Turns out that there was quite some overhead for new Aligner derivatives that can be aggregated in the base class, which is why I prioritized this.