houserjohn commented on PR #13914:
URL: https://github.com/apache/lucene/pull/13914#issuecomment-2666762847

   @HoustonPutman I can confirm that the latest commits fixed the exception in 
`testComputeDynamicNumericRangesWithLargeTopN` and the issue in 
`testComputeDynamicNumericRangesWithSameWeights`.
   
   Some of the randomized testing included in 
[GH#14238](https://github.com/apache/lucene/pull/14238) revealed some more bugs 
(they also revealed some of my own bugs in GH#14238):
   
   ```
   public void testComputeDynamicNumericRangesWithSameWeightsOutOfOrder() {
       List<DynamicRangeUtil.DynamicRangeInfo> expectedRangeInfoList = new 
ArrayList<>();
       long[] values =
           new long[] {
             20, 15, 59, 49, 13, 93, 72, 21, 36, 81, 57, 1, 90, 79, 16, 51, 7, 
17, 25, 63, 12, 5, 83,
             66, 48, 43, 55, 78, 64, 77, 65, 73, 80, 37, 54, 50, 95, 31, 97, 3, 
82, 29, 70, 26, 4, 46,
             34, 67, 87, 0, 30, 19, 41, 85, 84, 89, 8, 10, 22, 28, 6, 23, 88, 
40, 33, 44, 18, 27, 69,
             38, 91, 98, 62, 14, 35, 2, 92, 47, 94, 75, 32, 99, 86, 71, 74, 24, 
52, 96, 9, 58, 39, 76,
             56, 11, 53, 61, 42, 68, 60, 45
           };
       long[] weights =
           new long[] {
             50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 
50, 50, 50, 50, 50, 50,
             50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 
50, 50, 50, 50, 50, 50,
             50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 
50, 50, 50, 50, 50, 50,
             50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 
50, 50, 50, 50, 50, 50,
             50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50
           };
   
       // This is testComputeDynamicNumericRangesWithSameWeightsShuffled with 
seed
       // 9AE79D72C8DD56D8
       expectedRangeInfoList.add(new DynamicRangeUtil.DynamicRangeInfo(25, 
1250L, 0L, 24L, 12.0D));
       expectedRangeInfoList.add(new DynamicRangeUtil.DynamicRangeInfo(25, 
1250L, 25L, 49L, 37.0D));
       expectedRangeInfoList.add(new DynamicRangeUtil.DynamicRangeInfo(25, 
1250L, 50L, 74L, 62.0D));
       expectedRangeInfoList.add(new DynamicRangeUtil.DynamicRangeInfo(25, 
1250L, 75L, 99L, 87.0D));
       assertDynamicNumericRangeResults(values, weights, 4, 4950L, 5000L, 
expectedRangeInfoList);
   }
   ```
   Gives (look at **):
   ```
      >     java.lang.AssertionError: expected:<[DynamicRangeInfo[count=25, 
weight=1250, min=0, max=24, centroid=12.0], DynamicRangeInfo[count=25, 
weight=1250, **min=25**, max=49, centroid=37.0], DynamicRangeInfo[count=25, 
weight=1250, min=50, max=74, centroid=62.0], DynamicRangeInfo[count=25, 
weight=1250, min=75, max=99, centroid=87.0]]> but 
was:<[DynamicRangeInfo[count=25, weight=1250, min=0, max=24, centroid=12.0], 
DynamicRangeInfo[count=25, weight=1250, **min=43**, max=49, centroid=37.0], 
DynamicRangeInfo[count=25, weight=1250, min=50, max=74, centroid=62.0], 
DynamicRangeInfo[count=25, weight=1250, min=75, max=99, centroid=87.0]]>
      >         at 
__randomizedtesting.SeedInfo.seed([1E722DB63C0BE5AA:F241CBDDDDB3FF8C]:0)
      >         at junit@4.13.1/org.junit.Assert.fail(Assert.java:89)
      >         at junit@4.13.1/org.junit.Assert.failNotEquals(Assert.java:835)
      >         at junit@4.13.1/org.junit.Assert.assertEquals(Assert.java:120)
      >         at junit@4.13.1/org.junit.Assert.assertEquals(Assert.java:146)
      >         at 
org.apache.lucene.facet.range.TestDynamicRangeUtil.assertDynamicNumericRangeResults(TestDynamicRangeUtil.java:389)
      >         at 
org.apache.lucene.facet.range.TestDynamicRangeUtil.testComputeDynamicNumericRangesWithSameWeightsOutOfOrder(TestDynamicRangeUtil.java:172)
   ```
   I believe that this is still related to the minimum in a range bug. Note 
that this result is with the latest commits you added. Additionally, I think it 
might be helpful if you run some of these randomized tests overnight to reduce 
some of the back and forth. I'll post another comment later today with a 
modification of those randomization tests that you should be able to run.


-- 
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