Search for SwissExperiments doesn't return @Swiss-Experiments channel #154

Closed
opened 2019-05-20 19:32:42 +02:00 by tzarebczan · 2 comments
tzarebczan commented 2019-05-20 19:32:42 +02:00 (Migrated from github.com)

Swiss Experiments does work. We should take into account the - when searching.

`Swiss Experiments` does work. We should take into account the - when searching.
tiger5226 commented 2019-05-24 05:08:35 +02:00 (Migrated from github.com)

This is an interesting one. We can't really know that SwissExperiments should be split into Swiss Experiments. Elastic search does already recognize the - and will treat it as 2 words. Also our query will escape the reserved character so the actual text Swiss-Experiments will work fine due to the sloppy factor. Searching for Swiss or Experiments will also work fine because elastic search splits automatically on the -.

One possible solution is to take any claim name and remove any - and collapse the string into another field in the elastic document that is equally weighted to the current name field.

I am also open to other ideas here too if you have any.

This is an interesting one. We can't really know that `SwissExperiments` should be split into `Swiss Experiments`. Elastic search does already recognize the `-` and will treat it as 2 words. Also our query will escape the reserved character so the actual text `Swiss-Experiments` will work fine due to the sloppy factor. Searching for `Swiss` or `Experiments` will also work fine because elastic search splits automatically on the `-`. One possible solution is to take any claim name and remove any `-` and collapse the string into another field in the elastic document that is equally weighted to the current name field. I am also open to other ideas here too if you have any.
tiger5226 commented 2019-07-13 03:08:03 +02:00 (Migrated from github.com)

This is now fixed with the latest push to master. We have removed query washing for phrase matching and partial string comparison.

This is now fixed with the latest push to master. We have removed query washing for phrase matching and partial string comparison.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lighthouse.js#154
No description provided.