iamsanjay commented on PR #13198: URL: https://github.com/apache/lucene/pull/13198#issuecomment-2046550816
> @iamsanjay - thank you for working on this! I merged #12966, which should mean the original test failure is fixed. Do you want to verify that all is working as expected now? I tried running below test case which will only test two values [0,1], and it's failing. ``` public void testBoundaryCases() throws Exception { final float[] boundaryCases = new float[] {0, 1}; doTestDifferentValueRanges(() -> boundaryCases[random().nextInt(boundaryCases.length)]); } ``` You can also use this pull request, only remove the fix in check result method, to reproduce it. The idea is to check for only 0 and 1. Because in existing case we are getting values from random.nextFloat which makes it almost impossible to generate zeroes or ones. -- 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