[GitHub] [lucene] alessandrobenedetti commented on pull request #12314: Multi-value support for KnnVectorField

["alessandrobenedetti (via GitHub)" <gi...@apache.org>]

alessandrobenedetti commented on PR #12314:
URL: https://github.com/apache/lucene/pull/12314#issuecomment-1554825253

   I'll follow up with many clean up and tidy up on my own in the next few weeks.
   I should have a bit of bandwidth from now till Berlin Buzzword (mid June).
   Any feedback is welcome, I am pretty sure there are a *lot* of wrong/imperfect stuff around.
   I worked on this for many months, in my spare time, I probably spent most of my time merging from the main as this area changed a lot :)
   Didn't bother with pre-commit and stuff, I'll do it in the next few days.
   @tang-hi the idea is to allow multiple vectors in a field, sometimes you have multiple paragraphs and long texts you may want to split before encoding them to vectors, rather than concatenate everything to a single vector


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org