CoreWiki icon indicating copy to clipboard operation
CoreWiki copied to clipboard

Search feature

Open andrewcahill opened this issue 7 years ago • 13 comments
trafficstars

Should we handle multi word search but the words can be in any order. For example in the homepage we could not only find this by entering "default home" but also "home default". This would allow for a less strict search.

As well as that do you think we should also enable search after every key-up trigger, like in search panel i in visual studio solution explorer instead on needing to select search button.

andrewcahill avatar Jul 31 '18 17:07 andrewcahill

I think you're on to something, and we can update the SearchEngine by implementing the IArticlesSearchEngine interface with different providers and different search engine algorithms.

Let's look at this during a stream in the near future

csharpfritz avatar Aug 03 '18 16:08 csharpfritz

Requirements:

  • [ ] Should support fuzzy text search, not just a simple "like"

Breaking out autocomplete to a separate issue for tracking

csharpfritz avatar Aug 03 '18 17:08 csharpfritz

Sounds great.

andrewcahill avatar Aug 05 '18 16:08 andrewcahill

I suggest we start with a basic Lucene.NET implementation. That should be fairly straightforward to implement, loosely based on the interface we already have (we can extend it of course). Once we have tagging/labeling in place we can then extend it to include aggregate searches like Amazon has on left hand side. We can already do some of that like grouping by author. That's all basic Lucene out of the box. So is autocomplete btw. I've done implementations like that before, but I would wait until #255 is merged as that would give a clean business object to index.

bravecobra avatar Aug 06 '18 18:08 bravecobra

Hmm, why not make use of Azure Search for this? It brings Lucene and Fuzzy Search by default.

tiagodenoronha avatar Sep 20 '18 16:09 tiagodenoronha

Oh yea... that's DEFINITELY an option for a search implementation.

If folks don't have an Azure account, or maybe they have access to another provider I want to ensure that they can use Corewiki.

I would want to create a separate project with the Azure Search specific implementation in it. This way, we can optionally configure it

Jeff

On Thu, Sep 20, 2018 at 12:20 PM Tiago Noronha [email protected] wrote:

Hmm, why not make use of Azure Search for this? It brings Lucene and Fuzzy Search by default.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/csharpfritz/CoreWiki/issues/237#issuecomment-423244845, or mute the thread https://github.com/notifications/unsubscribe-auth/AAEy8UiTtadEyb3yj3Q_DlJSPxI1qVCBks5uc8BHgaJpZM4Vopzt .

csharpfritz avatar Sep 20 '18 16:09 csharpfritz

Sure, we can have x different implementations all implementing the same interface. Then just pick your poison.

  1. DB Search directly on the database
  2. Lucene directory
  3. Azure Search
  4. ElasticSearch

bravecobra avatar Sep 20 '18 17:09 bravecobra

Bingo!

I expect to start a "CoreWiki.Azure" project at Microsoft Ignite 2018, and put Azure-specific providers in that project.

Jeff

On Thu, Sep 20, 2018 at 1:01 PM bravecobra [email protected] wrote:

Sure, we can have x different implementations all implementing the same interface. Then just pick your poison.

  1. DB Search directly on the database
  2. Lucene directory
  3. Azure Search
  4. ElasticSearch

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/csharpfritz/CoreWiki/issues/237#issuecomment-423258523, or mute the thread https://github.com/notifications/unsubscribe-auth/AAEy8ccwF4hdJrlsBptG5nc9pudfrXl2ks5uc8nwgaJpZM4Vopzt .

csharpfritz avatar Sep 20 '18 17:09 csharpfritz

Gotcha.

So wait a bit until you create the project and then start accepting PR for it?

tiagodenoronha avatar Sep 20 '18 17:09 tiagodenoronha

@tiagodenoronha Have you looked more into this, i've started on Azure search integration, but have hit a wall, take a look if you like

ArgTang avatar Oct 13 '18 14:10 ArgTang

There’s a separate branch and project allocated for Azure integration.

Jeff

On Oct 13, 2018, at 10:05, ArgTang [email protected] wrote:

@tiagodenoronha Have you looked moer into this, ive started on Azure search integration, but have hit a wall, take a look if you like

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

csharpfritz avatar Oct 13 '18 14:10 csharpfritz

@jeff I saw that, and have merged and moved the relevant files into the AzureProject in my branch :)

ArgTang avatar Oct 13 '18 15:10 ArgTang

Glad to hear it, and can’t wait to take a further look at the PR

Jeff

On Oct 13, 2018, at 11:09, ArgTang [email protected] wrote:

@jeff I saw that, and have merged and moved the relevant files into the AzureProject in my branch :)

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

csharpfritz avatar Oct 13 '18 15:10 csharpfritz