benwtrent commented on PR #15356: URL: https://github.com/apache/lucene/pull/15356#issuecomment-3512549450
I am not gonna block this, but this list of "key features", and the repeating of concepts over and over again (e.g. Haversin Distance) smell of LLMs. LLMs continually over-repeat things because it's cheap for them. It's not for humans. Also, the `containsLine`, etc. should be comments on those methods themselves or something. I am not sure why every possible method is enumerated -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
