Closed hersle closed 2 months ago
This change uses div in the divrem function, rather than /.
div
divrem
/
This prevents integer divison from being promoted to floats, fixing the downstream issue JuliaSymbolics/Symbolics.jl#1083.
However, I see it breaks Multivariate gcd Rational{BigInt} tests, so I suppose it WIP now, at best.
Multivariate gcd Rational{BigInt}
Is this change viable to incorporate? Sorry, my knowledge of this library is very limited!
This change uses
div
in thedivrem
function, rather than/
.This prevents integer divison from being promoted to floats, fixing the downstream issue JuliaSymbolics/Symbolics.jl#1083.
However, I see it breaks
Multivariate gcd Rational{BigInt}
tests, so I suppose it WIP now, at best.Is this change viable to incorporate? Sorry, my knowledge of this library is very limited!