UpsilonNumworks/Upsilon
1
0
Fork 0
mirror of https://github.com/UpsilonNumworks/Upsilon.git synced 2026-03-22 15:20:39 +01:00
Code Issues Packages Projects Releases Wiki Activity

[apps/sequence/sequence] Replace sumBetweenBounds explicit definition by Poincare machinery

Browse Source
This commit is contained in:
Ruben Dashyan
2019-07-03 11:31:08 +02:00
committed by Léa Saviot
parent 70f96ee912
commit 4fef87ddb5
2 changed files with 3 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
apps/sequence/sequence.cpp
View File

@@ -6,6 +6,7 @@
#include <poincare/code_point_layout.h>
#include <poincare/vertical_offset_layout.h>
#include <poincare/integer.h>
#include <poincare/sum.h>
#include "../shared/poincare_helpers.h"
#include <string.h>
#include <cmath>
@@ -186,19 +187,8 @@ T Sequence::approximateToNextRank(int n, SequenceContext * sqctx) const {
}
double Sequence::sumBetweenBounds(double start, double end, Context * context) const {
double result = 0.0;
if (end-start > k_maxNumberOfTermsInSum || start + 1.0 == start) {
return NAN;
}
for (double i = std::round(start); i <= std::round(end); i = i + 1.0) {
/* When |start| >> 1.0, start + 1.0 = start. In that case, quit the
* infinite loop. */
if (i == i-1.0 || i == i+1.0) {
return NAN;
}
result += evaluateAtAbscissa(i, context);
}
return result;
Poincare::Sum sum = Poincare::Sum::Builder(expressionReduced(context).clone(), Symbol::Builder(UCodePointUnknownX), Poincare::Float<double>::Builder(start), Poincare::Float<double>::Builder(end)); // Sum takes ownership of args
return PoincareHelpers::ApproximateToScalar<double>(sum, *context);
}
Sequence::SequenceRecordDataBuffer * Sequence::recordData() const {