FrankChen021 commented on code in PR #12470: URL: https://github.com/apache/druid/pull/12470#discussion_r855718896
########## docs/development/extensions-core/stats.md: ########## @@ -35,9 +35,12 @@ The American Statistician, 37 (1983) pp. 242--247. variance = variance1 + variance2 + n/(m*(m+n)) * pow(((m/n)*t1 - t2),2) -where: - variance is sum(x-avg^2) (this is actually n times the variance) -and is updated at every step. - n is the count of elements in chunk1 - m is -the count of elements in chunk2 - t1 = sum of elements in chunk1, t2 = +where: + - variance is sum(x-avg^2) (this is actually n times the variance) +and is updated at every step. + - n is the count of elements in chunk1 + - m is the count of elements in chunk2 + - t1 = sum of elements in chunk1, t2 = Review Comment: Since all parameters are formatted as list item above, I think It's better to put `t2` in a separate list item too. -- 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]
