davecromberge opened a new pull request, #757: URL: https://github.com/apache/datasketches-java/pull/757
`CpcWrapper.getEstimate()`, `getLowerBound()` and `getUpperBound()` throw `SketchesStateException` for any empty sketch, so the class can't be used by a caller whose data may contain one. An empty image carries none of the high preamble fields. `EMPTY_HIP` sets the HIP flag, so `getEstimate()` takes the HIP branch and asks for `HIP_ACCUM`; `EMPTY_MERGED` takes the ICON branch and asks for `NUM_COUPONS`. Neither field is present, so both throw. Empty now reports zero coupons and a zero HIP accumulator, matching `CpcSketch`. Added `CpcWrapperEquivalenceTest`, which checks the wrapper against a heapified sketch on estimate, both bounds and lgK, across every lgK, the coupon counts that drive the flavor transitions, union results, and the three ways a sketch can be empty. -- 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]
