jina-ai / executors

internal-only
Apache License 2.0
31 stars 12 forks source link

FaissSearcher: /train should load data as well #282

Closed cristianmtr closed 2 years ago

cristianmtr commented 2 years ago

/train default, train a new model, replace old one, index data from training. After calling /train a user should be able to query directly on the data used for training, without further indexing.

/index add new data, don't retrain. If there is no model, emit warning, do not train.

This should be in-line with RiiSearcher https://github.com/jina-ai/internal-tasks/issues/217 See conversation