Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Terminal: recent command fuzzy search not very fuzzy #156325

Open
isidorn opened this issue Jul 26, 2022 · 13 comments
Open

Terminal: recent command fuzzy search not very fuzzy #156325

isidorn opened this issue Jul 26, 2022 · 13 comments
Assignees
Labels
feature-request Request for new features or functionality quick-pick Quick-pick widget issues
Milestone

Comments

@isidorn
Copy link
Contributor

isidorn commented Jul 26, 2022

Testing #156002

It seems like the terminal recent command fuzzy search wants to find fuzzy matches in different words only, if the matches are in the same word it ignores them.
For example, have a bunch of git commands. Turn on fuzzy search.
Search for "gt" - does not get matched against all git commands - which I would expect

Screenshot 2022-07-26 at 18 59 37

Screenshot 2022-07-26 at 18 59 50

@meganrogge
Copy link
Contributor

That is odd, though we're using the fuzzy searching that's used elsewhere in VSCode, so I wonder if there's an issue tracking that.

@meganrogge
Copy link
Contributor

@aeschli or @TylerLeonhardt I don't see one, but believe this is your domain

@TylerLeonhardt
Copy link
Member

@Tyriar didn't you implement the fuzziness here?

@meganrogge
Copy link
Contributor

Yes he did @TylerLeonhardt

@meganrogge meganrogge added bug Issue identified by VS Code Team member as probable bug terminal-shell-integration Shell integration infrastructure, command decorations, etc. labels Jul 27, 2022
@meganrogge meganrogge added this to the August 2022 milestone Jul 27, 2022
@meganrogge
Copy link
Contributor

I actually don't think @Tyriar had to do with this - we use matchesFuzzyIconAware which was added by @aeschli

@Tyriar
Copy link
Member

Tyriar commented Aug 22, 2022

This might be related to the quick pick's filtering, we should look into it first.

@Tyriar Tyriar modified the milestones: August 2022, September 2022 Aug 24, 2022
@Tyriar Tyriar modified the milestones: September 2022, Backlog Sep 23, 2022
@Fred-Vatin
Copy link

I confirm it happens filtering commands in palette too.

Typing pwsh doesn’t find any powershell commands.

powers returns…
image

pwsh returns nothing
image

My setup

Version: 1.73.1 (user setup)
Commit: 6261075646f055b99068d3688932416f2346dd3b
Date: 2022-11-09T04:27:29.066Z
Electron: 19.0.17
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: 10.2.154.15-electron.0
OS: Windows_NT x64 10.0.19044
Sandboxed: No
```

@gpeal
Copy link

gpeal commented Dec 4, 2022

I recently switched from the IntelliJ world to VSCode and the fuzzy match algorithm of VSCode is slowly driving me insane. In addition to the cases here, one thing that IntelliJ gets really right is case sensitive matching of partial words and priority for local symbols.

Check out this example. I am trying to match externalReport which is defined on the line immediately above. ext correctly matches. Typing R should increase the weight of the already-top result but instead, it drops off the list entirely.

The result of this is that I frequently wind up with random variables and imports as a result of the correct result disappearing after I type more letters that should only increase its weight.

CleanShot 2022-12-02 at 16 52 34@2x (1)

CleanShot 2022-12-02 at 16 52 30@2x (1)

You can see a similar result play out here:
CleanShot 2022-11-30 at 13 44 37@2x (1)
CleanShot 2022-11-30 at 13 45 20@2x (1)

This seems similar to
#34088
#131431

@Tyriar
Copy link
Member

Tyriar commented Dec 5, 2022

@gpeal this issue is specifically about ranking in the run recent terminal command. #34088 is probably what you want to comment on?

@Tyriar
Copy link
Member

Tyriar commented May 29, 2023

Assigning this to @TylerLeonhardt as it's essentially asking for proper fuzzy matching in the quick pick as in #34088

@Tyriar Tyriar added feature-request Request for new features or functionality quick-pick Quick-pick widget issues and removed bug Issue identified by VS Code Team member as probable bug labels May 29, 2023
@Tyriar Tyriar assigned TylerLeonhardt and unassigned Tyriar May 29, 2023
@TylerLeonhardt
Copy link
Member

@Tyriar but I thought you disabled filtering and implemented your own fuzzy logic?

@Tyriar
Copy link
Member

Tyriar commented May 29, 2023

@TylerLeonhardt I think that's just for contiguous search, for fuzzy we defer to the default

@TylerLeonhardt
Copy link
Member

ah so it's using fuzzy search...... it's just not fuzzy enough.

@Tyriar Tyriar removed the terminal-shell-integration Shell integration infrastructure, command decorations, etc. label Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for new features or functionality quick-pick Quick-pick widget issues
Projects
None yet
Development

No branches or pull requests

6 participants