[GitHub] [pinot] jasperjiaguo commented on pull request #9236: Add Support for Covariance Function

[GitBox <gi...@apache.org>]

jasperjiaguo commented on PR #9236:
URL: https://github.com/apache/pinot/pull/9236#issuecomment-1224910533

   > Can you please clarify or add a link in the description to the formula you are using to calculate covariance? I don't see "average value" being used in your calculation? It will also be good to add support for "covariance of sample" (which is probably more commonly used) along with "covariance of population"
   
   Yes. This should be simple if we extend `CovarianceAggregationFunction.java` and add Bessel's correction to `extract result`. We probably want to add that @SabrinaZhaozyf ?


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org