UpsilonNumworks/Upsilon
1
0
Fork 0
mirror of https://github.com/UpsilonNumworks/Upsilon.git synced 2026-01-19 00:37:25 +01:00
Code Issues Packages Projects Releases Wiki Activity

[poincare] Remove wrong assert and comment

Browse Source
This commit is contained in:
Émilie Feral
2019-08-16 16:42:58 +02:00
parent fbbc11cf33
commit f6ddc005b6
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
poincare/src/expression.cpp
View File

@@ -536,7 +536,6 @@ void makePositive(Expression * e, bool * isNegative) {
}
void Expression::beautifyAndApproximateScalar(Expression * simplifiedExpression, Expression * approximateExpression, ExpressionNode::ReductionContext userReductionContext, Context * context, Preferences::ComplexFormat complexFormat, Preferences::AngleUnit angleUnit) {
assert(type() != ExpressionNode::Type::Matrix);
/* Case 1: the reduced expression is ComplexCartesian or pure real, we can
* take into account the complex format to display a+i*b or r*e^(i*th) */
if (type() == ExpressionNode::Type::ComplexCartesian || isReal(context)) {
@@ -598,10 +597,10 @@ void Expression::simplifyAndApproximate(Expression * simplifiedExpression, Expre
return;
}
// Step 2: we approximate and beautify the reduced expression
// si matrix à la racine, appeler simplifyAndApproximateOfScalar sur chaque entrée de la matrice, sinon appeler simplifyAndApproximateOfScalar de toi même
/* Case 1: the reduced expression is a matrix: We scan the matrix children to
* beautify them with the right complex format. */
if (e.type() == ExpressionNode::Type::Matrix) {
// TODO: this method enables to take the complex format into account when the result is a matrix of scalar. It won't work for nested matrices... Find a more elegant and general solution?
Matrix m = static_cast<Matrix &>(e);
*simplifiedExpression = Matrix::Builder();
if (approximateExpression) {