Skip to content
This repository has been archived by the owner on Jan 30, 2023. It is now read-only.

Commit

Permalink
src/sage/arith/misc.py: Use sage.rings.abc instead of importing eleme…
Browse files Browse the repository at this point in the history
…nt classes RealNumber, ComplexNumber
  • Loading branch information
Matthias Koeppe committed Sep 30, 2021
1 parent 4e2f0ab commit 5145d43
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions src/sage/arith/misc.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,7 @@
from sage.rings.integer_ring import ZZ
from sage.rings.integer import Integer, GCD_list
from sage.rings.rational import Rational
from sage.rings.real_mpfr import RealNumber
from sage.rings.complex_mpfr import ComplexNumber
from sage.rings.abc import RealField, ComplexField

from sage.rings.fast_arith import arith_int, arith_llong, prime_range

Expand Down Expand Up @@ -203,7 +202,7 @@ def algdep(z, degree, known_bits=None, use_bits=None, known_digits=None,
return None
return z.denominator()*x - z.numerator()

if isinstance(z, (RealNumber, ComplexNumber)):
if isinstance(z.parent(), (RealField, ComplexField)):

log2_10 = math.log(10,2)

Expand Down Expand Up @@ -3609,7 +3608,7 @@ def binomial(x, m, **kwds):
return P(x.binomial(m, **kwds))

# case 3: rational, real numbers, complex numbers -> use pari
if isinstance(x, (Rational, RealNumber, ComplexNumber)):
if isinstance(x, Rational) or isinstance(P, (RealField, ComplexField)):
return P(x.__pari__().binomial(m))

# case 4: naive method
Expand Down

0 comments on commit 5145d43

Please sign in to comment.