benwtrent commented on PR #12694: URL: https://github.com/apache/lucene/pull/12694#issuecomment-1770816807
> ZERO_EXTEND_B2I and ZERO_EXTEND_B2S instead of adding branches to the code and AND instructions. Thank you! > I don't think we should 'add' unsigned vectors format, if it is better we should change to it and remove the signed format. We have to maintain all this stuff. This is tricky as folks who give Lucene `byte[]` vectors now expected signed operations. While this isn't an issue with euclidean, it is an issue with dot_product, etc. Wouldn't it be a breaking change to adjust how scoring works? -- 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-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
