From c09056b98a98b75a69b87caacc0c03e561c0cddf Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=89milie=20Feral?= Date: Tue, 23 Oct 2018 17:20:19 +0200 Subject: [PATCH] [shared] StorageCartesianFunction: sumBetweenBounds does not work, add a comment to explain why --- apps/shared/storage_cartesian_function.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/apps/shared/storage_cartesian_function.cpp b/apps/shared/storage_cartesian_function.cpp index 29adc3f5e..191aefe22 100644 --- a/apps/shared/storage_cartesian_function.cpp +++ b/apps/shared/storage_cartesian_function.cpp @@ -93,6 +93,7 @@ double StorageCartesianFunction::approximateDerivative(double x, Poincare::Conte } double StorageCartesianFunction::sumBetweenBounds(double start, double end, Poincare::Context * context) const { + // TODO: this does not work yet because integral does not understand UnknownX Poincare::Integral integral(expression(context).clone(), Poincare::Float(start), Poincare::Float(end)); // Integral takes ownership of args /* TODO: when we approximate integral, we might want to simplify the integral * here. However, we might want to do it once for all x (to avoid lagging in