Adds CostDigest t-digest data structure to the pricing package#42
Conversation
Signed-off-by: David Breitgand <davidbreitgand@users.noreply.github.com>
| @@ -0,0 +1,99 @@ | |||
| /* | |||
There was a problem hiding this comment.
Since these test verifies that the library clone() works as expected they should not be included here, and should be removed.
There was a problem hiding this comment.
I removed the tests for now. They probably will creep back as CostDigest will include a warmup/exploration counter. But we can defer this discussion.
|
Hi @ronenkat , @Mohammad-nassar10 , the follow-up work for this feature is ready. Since it builds directly on top of this branch, I have it staged in my fork for visibility and allow for your comments: requestcostmetadata. Thank you. |
Signed-off-by: David Breitgand <davidbreitgand@users.noreply.github.com>
|
@davidbreitgand please fix the CI error |
|
Lets merge after it passes CI |
|
/lgtm |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
2 participants
What type of PR is this?