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

Search does Not work #5540

Closed
ondejcik opened this issue Oct 16, 2015 · 7 comments
Closed

Search does Not work #5540

ondejcik opened this issue Oct 16, 2015 · 7 comments
Assignees
Labels

Comments

@ondejcik
Copy link

Hi,

i am testing new dynamo 0.9.1 (DynamoInstall0.9.1.20151015T0049)
and according to descriptopn it should have improoved search, but it does not work at all:

i am looking for node (id to element) from archilab package that is saved on disk - external path:

here it is in browser

id

here is amazingly long list with very little relevant nodes (to search), and further one i am after is nowhere in 4 page long list:

id to element

@andydandy74
Copy link
Contributor

This is still an issue in 1.0 - I find this very disruptive to the overall workflow...
suckysearch

@jnealb
Copy link
Collaborator

jnealb commented May 6, 2016

@andydandy74 @pboyer @Racel @kronz Agreed; work in progress.

@andydandy74
Copy link
Contributor

If the search term matches a node name exactly, why not display that node as the first result? That should be an easy fix that could be quickly accomplished. Please, please. please give us at least that.
Examples this could be tested on in current versions (and that drive me mad time and time again):

  • "not" (shows RemoveIfNot instead of Not as best result)
  • "sum" (shows Add instead of Sum as best result)
  • ...

CC @kronz @Racel @jnealb

@smangarole
Copy link
Contributor

@Racel @kronz Can you please provide an update on this issue?

@kronz
Copy link
Contributor

kronz commented Sep 25, 2017

Search is a big project, and we want to get it right. We have major work slated to do on this after the 2.0 release, and we'd like to start using analytics to really nail it. In the meantime, there are loads of very small changes you correctly point out. We have avoided getting distracted by these, and perfect may be the enemy of good in this case, but we are trying to get to a better systematic overall solution rather than spot fixes.

@johnpierson
Copy link
Member

Partial progress at, QNTM-5869

@Racel
Copy link
Contributor

Racel commented Dec 19, 2018

The issue brought up by @ondejcik and @andydandy74 required a fix to a search algorithm that made queries with whitespaces difficult to manage. We have corrected this issue in these PRs #9332 and #9271 which were recently merged into master. The reported issue should now be resolved. Thanks so much for your feedback and patience. This fix will appear in the next release.

@Racel Racel closed this as completed Dec 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

8 participants