ajwerner

> The question here should be whether there should be a default limit and if so whether it should be 4 MiB. I don't know if this is worth having...

The returned stream type is generic, that means that it can be whatever you want. One thing that you can do is move the join handles into the returned stream...

Good catch, that's pretty egregious.

Not that explaining the source of bugs is all that valuable, the source of this bug is that I ported `td_trimmed_mean` back from https://github.com/ajwerner/tdigest/blob/9165f388a6b2c88703b9630c9d15e59d2d212d7e/internal/tdigest/tdigest.go#L17. In the go repo, which I've...

I'm less convinced that this one is a bug. I think this interpolation is pretty sane. Let's work through a small example. For shorthand a centroid is `{mean, count}`. Say...

Again, I do appreciate the interest. The logic to merge centroids of the same value is missing in my go implementation and I'll add it. Thanks for bringing it. I...

Doesn't look like it.

That makes sense. I now see what I'm trying to do with the summing wouldn't make sense for the left centroid in quantile_of and the current just pick whichever one...

That's another great point. If we know we have only one point then our interpolation function could take advantage of that. This assumption will only work if you know you're...

> That essentially contain the same amount of information (at least it seems to me). Why'd you use 10 for the first one, but 20 for the second one? An...