jtibshirani commented on pull request #315: URL: https://github.com/apache/lucene/pull/315#issuecomment-929588721
I'm not an expert in designing formats, but this looks good to me overall! I left some minor comments. > EG I saw Julie changed the default beam-width to 100, which is fine, makes sense for the public datasets, but I've observed the 32 we initially had as being a better choice for some of the vectors we work with. That's interesting to know, thanks for sharing. As a clarification, the default beamWidth used to be 16. -- 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
