tisonkun opened a new pull request, #255: URL: https://github.com/apache/datasketches-rust/pull/255
## Summary - define `estimated_size()` as the sketch value's inline size plus retained capacity of its internal heap-backed containers - keep every existing method unconditionally available and explicitly make accounting shallow for generic items, summaries, and policies - distinguish in-memory estimates from serialized size and exclude allocator bookkeeping and temporary allocations - align the documentation of all sketches, compact forms, and stateful operators with the shared contract - add regression coverage showing that nested `String` and `Vec<f64>` allocations do not affect generic sketch estimates The existing calculations already followed this model, so this patch does not change API signatures or returned values. No changelog entry is needed because this clarifies the existing contract without changing observable behavior. ## Validation - `cargo x check` - `cargo x test` - `cargo x lint` Closes #193. -- 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]
