Skip to content

More like this#747

Merged
carvil merged 2 commits intomasterfrom
more-like-this
Feb 13, 2017
Merged

More like this#747
carvil merged 2 commits intomasterfrom
more-like-this

Conversation

@carvil
Copy link
Copy Markdown
Contributor

@carvil carvil commented Feb 13, 2017

We are in the process of trying to populate the Related Item's component (or similar component) with links that are similar to a given content item based on the "more like this" feature in ElasticSearch.

This PR brings that functionality into Rummager in order to let us experiment with it in the new navigation.

Trello: https://trello.com/c/euH6zl1h/393-re-introduce-more-like-this-functionality-into-rummager

Part of: https://trello.com/c/HjuOpRCc/231-as-a-user-i-want-relevant-links-in-the-sidebar-so-i-can-move-sideways-or-zoom-out-orient-myself

This work was started by @rubenarakelyan some time ago.

Ruben Arakelyan and others added 2 commits February 10, 2017 11:49
This commit adds support in rummager for “more like this” queries to be sent to Elasticsearch. “More like this” queries allow searching using an existing indexed document as a starting point, and returning a set of documents that are “similar” to it content-wise.

A “more like this” search can be carried out using the existing rummager search endpoint with the new `similar_to` option, which takes a base path as its value (more correctly, the ID of the document in the Elasticsearch index). These searches cannot be combined with a text query (using the `q` option) and they also cannot be custom sorted (they are already sorted by similarity).
There has been changes in the tests' setup recently. This commit makes
sure the tests related to "more like this" queries are up-to-date.
@carvil carvil merged commit 5ccd756 into master Feb 13, 2017
@carvil carvil deleted the more-like-this branch February 13, 2017 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants