Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 3882 by trel...@psu.edu: divisors() is not type-consistent
http://code.google.com/p/sympy/issues/detail?id=3882

Ran into a bug today with divisors()

The issue is that divisors() is not consistent in the type of the factors it returns. If it is passed an int, then all returned factors are ints. But if it is passed something of type sympy.core.numbers.Integer, then the returned list can contain int's and sympy.core.numbers.Integer's. This leads to something I found real confusing at first...

In [57]: set(divisors(4)).union(set(divisors(numbers.Integer(2))))
Out[57]: set([1, 2, 2, 4])

There are several simple ways to fix this. But are there any particular conventions that apply? Should all the factors have the same type as the original? Or should we escalate the type of all of the factors returned?


--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"sympy-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to sympy-issues+unsubscr...@googlegroups.com.
To post to this group, send email to sympy-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/sympy-issues?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to