doesn't always list the closest matching first

Bug #930065 reported by Sebastien Bacher
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Application Menu Indicator
Fix Released
Medium
Ted Gould
indicator-appmenu (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Using current precise (i.e 0.3.90-0ubuntu1) with gedit in french:

- I type "pré" in the hud to go to "préférences"

- the choises that are listed are:
* Fichier > aperçu avant impression
* Connexion au réseau...
* Edition > préférences

I think the third one should be listed first since it does exact match what I'm typing where the others don't

Tags: hud

Related branches

tags: added: hud
Revision history for this message
Lars Karlitski (larsu) wrote :

I had a comment about search quality when reviewing part of the hud code[1]. Will paste it here again so that it doesn't get lost:

imo, sorting by summing the relative usage and distance isn't a very good
strategy, as both usages and distances will tend to only have few very
small and many very large values, which is always tricky when dealing
with percentages.

Somewhat exaggerated example: Usage count for "open" and "save" is 10 each.
I type "blur" (which is a perfect match) for the first time:

       distance usage total (as in usage_sort())
blur 0 0 % 0 0 % 100 %
open 4 50 % 10 50 % 100 %
save 4 50 % 10 50 % 100 %

Ranking those 3 the same is surely not what we want?

[1] https://code.launchpad.net/~ted/indicator-appmenu/hud/+merge/89905/comments/194449

Revision history for this message
Ted Gould (ted) wrote :

@lars, I think that's a different bug #923999

I added a test case to the distance suite (in the attached branch) and that fails as well. Confirming.

Changed in indicator-appmenu:
status: New → Confirmed
importance: Undecided → Medium
Changed in indicator-appmenu (Ubuntu):
status: New → Triaged
importance: Undecided → Medium
Ted Gould (ted)
Changed in indicator-appmenu:
status: Confirmed → In Progress
assignee: nobody → Ted Gould (ted)
milestone: none → 0.3.91
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package indicator-appmenu - 0.3.90-0ubuntu2

---------------
indicator-appmenu (0.3.90-0ubuntu2) precise; urgency=low

  * Backport upstream work for better matching of utf8 strings (lp: #930065)
 -- Sebastien Bacher <email address hidden> Tue, 14 Feb 2012 17:58:13 +0100

Changed in indicator-appmenu (Ubuntu):
status: Triaged → Fix Released
Ted Gould (ted)
Changed in indicator-appmenu:
status: In Progress → Fix Committed
Ted Gould (ted)
Changed in indicator-appmenu:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.