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

Reply via email to