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

Fuzzy search to open file doesn't show most relevant suggestion first #4770

Closed
andersekdahl opened this issue Mar 29, 2016 · 18 comments
Closed
Assignees
Labels
verified Verification succeeded
Milestone

Comments

@andersekdahl
Copy link

  • VSCode Version: 0.10.11
  • OS Version: Windows 8.1

fuzzy
As you see in the image, I've searched for overlay/index but the file I'm looking for is not the first suggestion even if there is only one file whose path fully matches what I've written. Changing / to \ doesn't seem to matter.

Both Atom and Sublime have really great implementations of this, would be great to have it in VS Code as well.

@bpasero
Copy link
Member

bpasero commented Mar 30, 2016

@andersekdahl I would be interested in getting access to the workspace for me to reproduce this, if possible?

@bpasero bpasero added the bug Issue identified by VS Code Team member as probable bug label Mar 30, 2016
@bpasero bpasero added this to the April 2016 milestone Mar 30, 2016
@bpasero bpasero self-assigned this Mar 30, 2016
@andersekdahl
Copy link
Author

I don't think that possible unfortunately. I'll see if I can create an isolated workspace to reproduce it and share that.

@bpasero
Copy link
Member

bpasero commented Mar 30, 2016

@andersekdahl thanks. I just pushed a fix for our scorer and was wondering if it fixes this issue too. That would help me to verify quickly.

@andersekdahl
Copy link
Author

I can't seem to reproduce it in another workspace. Can I get access to a build with your fix to test if it solves the issue?

@andersekdahl
Copy link
Author

Is it this commit 9147160? Building from source on that commit now.

@bpasero
Copy link
Member

bpasero commented Mar 30, 2016

@andersekdahl yes, that one will boost scores higher for consecutive matches. I would assume "Overlay" to be #1 after that change.

@andersekdahl
Copy link
Author

Can't build from source unfortunately, I have VS 2015 but I'm missing cl.exe and don't have a VS setup file to run to install it. Could you comment on this issue when your commit is in a public release?

@bpasero
Copy link
Member

bpasero commented Mar 30, 2016

@andersekdahl building a version of it now that I can share to you.

@andersekdahl
Copy link
Author

That seems to fix it, thanks!

There are still some other things though. Typing overlayindex (without slash) gives me the same behaviour as the image I posted. And searching for just overlay gives me matches for two files named overlay.xxx but no match at all for files in the folder Overlay. The search seems to heavily promote search hits in file names and not the paths.

@bpasero
Copy link
Member

bpasero commented Mar 30, 2016

@andersekdahl that is totally true yeah. actually I am running the scorer first over the file name and then over the file path to naturally promote file matches over path matches.

@bpasero
Copy link
Member

bpasero commented Mar 30, 2016

@andersekdahl I am still wondering why you would not get any match in "overlay" though for files that live in that folder.

E.g. in this workspace I have a folder "lib" and can search for files within:

image

@andersekdahl
Copy link
Author

Yeah it's hard to tell. Like I said, I couldn't reproduce it in a test workspace. Could it have something to do with the size of the workspace?

@bpasero
Copy link
Member

bpasero commented Mar 30, 2016

@andersekdahl good point, there is an actual limit of results in quick open.

@bpasero
Copy link
Member

bpasero commented Mar 30, 2016

Marking as closed, given the initial report seems fixed.

@bpasero bpasero closed this as completed Mar 30, 2016
@bpasero bpasero removed the bug Issue identified by VS Code Team member as probable bug label Mar 30, 2016
@andersekdahl
Copy link
Author

Fine by me, I'll see if I can create test cases for the other things I mentioned and open new issues if I can.

@andersekdahl
Copy link
Author

Thanks for the quick replies btw!

@bpasero
Copy link
Member

bpasero commented Mar 30, 2016

np 👍

@bpasero bpasero added the verified Verification succeeded label Apr 28, 2016
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

2 participants