Closed kpeeters closed 1 year ago
The simplification of 1/(-a) led to an issue where the rational 1/-1 was not simplified to -1. This was due to a missing canonicalize call after generating the mpq_class with arguments (1, -1). This fixes the issue reported in https://cadabra.science/qa/2378/behavior-crashes-substituting-expression-depending-expression
canonicalize
mpq_class
(1, -1)
The simplification of 1/(-a) led to an issue where the rational 1/-1 was not simplified to -1. This was due to a missing
canonicalizecall after generating thempq_classwith arguments(1, -1). This fixes the issue reported in https://cadabra.science/qa/2378/behavior-crashes-substituting-expression-depending-expression