Skip to content
This repository has been archived by the owner on Jan 28, 2020. It is now read-only.

Search doesn't find phrases #866

Open
pwilkins opened this issue Jan 21, 2016 · 4 comments
Open

Search doesn't find phrases #866

pwilkins opened this issue Jan 21, 2016 · 4 comments
Labels

Comments

@pwilkins
Copy link
Contributor

If you enter the title of a learning resource into the search bar, you'd expect the learning resource with that title to appear near the top of the resultset. Search for "Astronauts Playing Catch", the title of a problem. It comes back on the seventh page. http://lore.odl.mit.edu/repositories/classical-mechanics/?q=Astronauts+Playing+Catch&page=7

I believe there are two issues here, one hiding the other. The first issue may be that the index doesn't contain word proximity information. The second issue, hidden by the first, is that the ranking of results doesn't properly influence the display order of learning resources.

Or it's something else entirely. :-)

@pdpinch @giocalitri

@pdpinch
Copy link
Member

pdpinch commented Jan 21, 2016

Does the sort order default to "Number of Views" ?

There is no view data, so is there a secondary sort order that we're using?

@noisecapella
Copy link
Contributor

I think the secondary sort is by id, which is basically creation order

@pwilkins
Copy link
Contributor Author

I found this article on the Elasticsearch blog which discusses querying on phrases:
https://www.elastic.co/blog/found-text-analysis-part-1#optimizing-phrase-searches-with-shingles

The article validates what @giocalitri said in our f2f meeting the other day.

@pwilkins
Copy link
Contributor Author

As @giocalitri warned, this may get complicated.

Setting the default sort order to "relevance" moves the learning resource titled "Astronauts playing catch" to the top of the result list, but adding "Astronauts playing catch" to the description of another LR bumps the titled LR to second spot.

screen shot 2016-01-22 at 2 04 01 pm

Since I'm doing this locally, the fact that there are no LR descriptions (other than the one I added) may have unbalanced its relevance.

@Ferdi Ferdi added the Backlog label Apr 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants