Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Scale method to QuantileDigest #1053

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

skrzypo987
Copy link

No description provided.

Co-authored-by: Trong Nguyen <trongdn@fb.com>
@martint
Copy link
Member

martint commented Mar 29, 2023

What’s the use case for this?

@skrzypo987
Copy link
Author

What’s the use case for this?

Adding a scale method to Trino. We can do it in Trino, but I thought this is the proper place

@martint
Copy link
Member

martint commented Mar 29, 2023

Understood, but what's the use case for having a scale method?

@Praveen2112
Copy link
Contributor

The purpose of the scale method is to scale up or down QDigest summary for a given set of data. Say if we have a summary of an order delivered from point A to point B stored in QDigest, the scale function allows us to extrapolate if the point A or point B diverges or converges.

@Praveen2112
Copy link
Contributor

@martint Do we need any additional information here ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants