fuzzy and fuzzyBy produce false positives on long strings (too fuzzy?)
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 55/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Stale
- Tech stack
- angularjs, javascript
Research direction
The issue is in the fuzzy and fuzzyBy filters. Look at the filter implementation files, likely named fuzzy.js or similar. The test case shows the bug: a long string incorrectly matches 'foobar'. Start by understanding the matching algorithm, then adjust the threshold or scoring for long strings. Run the existing tests to ensure changes don't break existing functionality.
Written by the indexing model from the issue text.
Description
While testing the fuzzy and fuzzyBy filters on some sample text, I noticed they tend to match any search term for long string data. The longer the text string, the more likely it will match on words that barely resemble the search string. This makes it hard to use effectively, since the filtered results have a very low signal-to-noise ratio.
var $filter = angular.element(document.body).injector().get('$filter');
var arr = [
{ text: 'foo bar' },
{ text: 'Curabitur ultricies metus eu neque luctus, vitae efficitur ' +
'quam elementum. Aliquam convallis diam et pretium placerat. Morbi ' +
'tellus turpis, sollicitudin eu nunc et, mollis sodales lectus. ' +
'Pellentesque lacinia augue id risus condimentum bibendum.'
}
];
console.log($filter('fuzzy')(arr, 'foobar').length); // returns 2
- Dominant language
- JavaScript
- Stars
- 2.9k
- Forks
- 319
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from a8m/angular-filter
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
a8m/angular-filter#267 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
a8m/angular-filter#260 · 1 reaction ·
-
KiB and MiB are wrongly used in place of kB and MBPossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
a8m/angular-filter#254 · 2 comments · 1 reaction ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
a8m/angular-filter#250 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
a8m/angular-filter#208 · 1 comment · 1 reaction ·
All issues in a8m/angular-filter
Similar issues
-
Engineering
Difficulty 2/5 1-3 hours Newbie friendliness 66/100
techmatters/terraso-web-client#3095 ·
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
Service process inherits the caller's cwd at first use, holding that folder open on Windows (EBUSY)Open
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
nextcloud/viewer#3424 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
Maintainers usually reply within 1 day