PlayniteExtensionsCollection icon indicating copy to clipboard operation
PlayniteExtensionsCollection copied to clipboard

Fixed 2 games both with empty collections resulting in 0 similarity

Open Jeshibu opened this issue 4 months ago • 3 comments

This fixes #469

I have verified that:

  • [x] These changes work, by building the extension and testing.
  • [x] That the changes comply with the rules indicated in the repository.
  • [x] Pull request is targeting master branch.

Jeshibu avatar Feb 18 '24 19:02 Jeshibu

Changes like this should be done in the control game matching algorithm and not to that method, since it calculates the Jaccard Similarity and this change would make it inaccurate. I'm also not sure if the best approach is to give a fixed number in such cases but I'm open to suggestions to improve the algorithm and if you've checked that it works fine.

darklinkpower avatar Feb 25 '24 02:02 darklinkpower

I've changed it so that it only uses fields that have values for at least one of the games. Fields with 0 (non-ignored) items in both lists now do not contribute to the similarity score, but also don't contribute to the minimum similarity score threshold.

I've also tweaked the minimum similarity threshold, but I want to put that in an advanced settings tab, along with the field weights, if you think this is the right way to go.

Jeshibu avatar Feb 25 '24 11:02 Jeshibu

Added settings. If this is about right I'll add localization too.

Jeshibu avatar Mar 23 '24 10:03 Jeshibu