We need to add insert and delete too

We need to add insert and delete too

To be more specific, lets use batch_insert() and batch_delete().

@mdouze @ingberam I went ahead and prototyped the new/updated candidate virtual methods in BaseANN. See the Files Changed tab for the proposals.

@sourcesync Thanks so much for going ahead with this! As I understood, there is no hybrid query, but only a dense query, add, remove (possible with metadata) and a sparse query. Given these different settings, I think it makes more sense to have BaseDenseANN and BaseSparseANN subclasses from BaseANN. Also, there are probably going to be different runners for the different scenarios since I imagine participants targeting single scenarios (and maybe they want to use their own runner and not provide a wrapper.)

My plan was to merge the baselines next week and then refactor the code around them. It would be great if we could join forces @sourcesync

@sourcesync Thanks so much for going ahead with this! As I understood, there is no hybrid query, but only a dense query, add, remove (possible with metadata) and a sparse query. Given these different settings, I think it makes more sense to have BaseDenseANN and BaseSparseANN subclasses from BaseANN. Also, there are probably going to be different runners for the different scenarios since I imagine participants targeting single scenarios (and maybe they want to use their own runner and not provide a wrapper.)

My plan was to merge the baselines next week and then refactor the code around them. It would be great if we could join forces @sourcesync

Awesome @Martin Aumüller. Yeah, I think approaching this with different base classes is a great choice esp. if "hybrid" is not a part of this competition.

( Note that if you already have a branch / PR going, I'm quite happy to close this draft PR...)