core icon indicating copy to clipboard operation
core copied to clipboard

lowering recall threshold

Open scicco opened this issue 5 months ago • 1 comments

Description

This is my first attempt to try to contribute a bit to this awesome project😹 This commit will lower the recall threshold. It also separates the config introducing the RecallSettings class as suggested in TODO

I've set the following constant values:

    DEFAULT_K = 3
    DEFAULT_TRESHOLD = 0.5

please let me know if those values are appropriate.

Related to issue (#909)

Type of change

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ ] This change requires a documentation update

Checklist:

  • [x] My code follows the style guidelines of this project
  • [x] I have performed a self-review of my own code
  • [x] I have commented my code, particularly in hard-to-understand areas

scicco avatar Sep 16 '24 22:09 scicco