From c0118af31517407fdb14a73cf0229ba553b57781 Mon Sep 17 00:00:00 2001 From: Ozzie Gooen Date: Tue, 5 Apr 2022 15:34:19 -0400 Subject: [PATCH] Changed warning comment --- .../Distributions/PointSetDist/AlgebraicShapeCombination.res | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/squiggle-lang/src/rescript/Distributions/PointSetDist/AlgebraicShapeCombination.res b/packages/squiggle-lang/src/rescript/Distributions/PointSetDist/AlgebraicShapeCombination.res index d94bd4a5..c0d85e60 100644 --- a/packages/squiggle-lang/src/rescript/Distributions/PointSetDist/AlgebraicShapeCombination.res +++ b/packages/squiggle-lang/src/rescript/Distributions/PointSetDist/AlgebraicShapeCombination.res @@ -118,7 +118,7 @@ let combineShapesContinuousContinuous = ( | #Logarithm => (m1, m2) => log(m1) /. log(m2) } // note: here, mInv2 = mean(1 / t2) ~= 1 / mean(t2) - // TODO: I don't know what the variances are for exponentatiation or logarithms + // TODO: Variances are for exponentatiation or logarithms are almost totally made up and very likely very wrong. // converts the variances and means of the two inputs into the variance of the output let combineVariancesFn = switch op { | #Add => (v1, v2, _, _) => v1 +. v2