tisonkun commented on PR #210: URL: https://github.com/apache/datasketches-rust/pull/210#issuecomment-5393897283
@ariesdevil I took a closer look at the mutable storage state machine and opened a small PR against your `t-digest` branch: https://github.com/ariesdevil/datasketches-rust/pull/1 It represents the initial staging buffer and active centroid layout as enum variants, so the two allocations cannot coexist and `num_unmerged` only exists with active centroid storage. The update thresholds, stable merge order, serialized bytes, and allocation behavior remain unchanged. I ran the full check/test/lint suite, byte-for-byte differential scenarios, and targeted TDigest benchmarks. Please take a look if this direction makes sense to you. -- 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]
