Updates:
Status: Invalid
Cc: asmeu...@gmail.com
Comment #27 on issue 1467 by skirpic...@gmail.com: sorting values with
imaginary numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
Closed.
Ordering for complex doesn't make sense (see issue 2625) and for so
Updates:
Labels: -Milestone-Release0.7.2
Comment #26 on issue 1467 by vlada.pe...@gmail.com: sorting values with
imaginary numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
What is the actual issue here? Obviously we cannot sort anything in the
complex plane, i
Comment #24 on issue 1467 by matt...@gmail.com: sorting values with
imaginary numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
btw. For sorting expressions we have sort_key, e.g.:
In [1]: sorted([3, -I, I, 2 + I], key=lambda expr: sympify(expr).sort_key())
Out[1]: [3,
Updates:
Labels: -Milestone-Release0.7.0 Milestone-Release0.7.1
Comment #23 on issue 1467 by asmeurer: sorting values with imaginary
numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
(No comment was entered for this change.)
--
You received this message because
Updates:
Labels: -NeedsReview NeedsBetterPatch
Comment #21 on issue 1467 by Vinzent.Steinberg: sorting values with
imaginary numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
I think Ondrej was not fine with the current patch due to performance
implications.
--
Comment #20 on issue 1467 by Vinzent.Steinberg: sorting values with
imaginary numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
In my opinion, comparison (and sorting) of non-reals should fail. For
canonical
sorting we should create our own function. Something like:
>>
Comment #19 on issue 1467 by asmeurer: sorting values with imaginary
numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
OK, so I was looking for sorted([3, I]) instead of sorted([3, sqrt(I)]).
Based on a quick git bisect, the second one
was fixed with this commit:
b3f7e
Comment #18 on issue 1467 by asmeurer: sorting values with imaginary
numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
I tried a bunch of commits all the way back to the first one in the
repository, and they all give:
In [1]: sorted([3,I])
Out[1]: [3, I]
So I think th
Comment #17 on issue 1467 by smichr: sorting values with imaginary numbers
in radical
http://code.google.com/p/sympy/issues/detail?id=1467
The original issue no longer causes a problem in the master branch...not
sure what
fixed it.
###
C:\DOCUME~2\chris\sympy\sympy>\python26\python.exe
Pyth
Comment #15 on issue 1467 by ondrej.certik: sorting values with imaginary
numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
What is the status of this patch --- does it need improvements?
--
You received this message because you are listed in the owner
or CC fields of th
Comment #14 on issue 1467 by smichr: sorting values with imaginary numbers
in radical
http://code.google.com/p/sympy/issues/detail?id=1467
We chatted about this on irc and in light of their not being a well defined
comparison test for a real and imaginary that it should be removed so
sorted(
Comment #13 on issue 1467 by asmeurer: sorting values with imaginary
numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
Yes, it is mathematically impossible to have a well defined ordering on
complex numbers. See
http://en.wikipedia.org/wiki/Ordered_field. I should als
Updates:
Labels: -PassedReview NeedsReview
Comment #12 on issue 1467 by ondrej.certik: sorting values with imaginary
numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
Many thanks for the patch and sorry it took my so long to review.
I think __lt__ should not retu
Comment #11 on issue 1467 by smichr: sorting values with imaginary numbers
in radical
http://code.google.com/p/sympy/issues/detail?id=1467
Ted, regarding the underlying issues:
I.is_comparable is False because I derives from Atom, not NumberSymbol. If
you change
it to be comparable then tes
Updates:
Labels: -NeedsReview PassedReview
Comment #10 on issue 1467 by smichr: sorting values with imaginary numbers
in radical
http://code.google.com/p/sympy/issues/detail?id=1467
I am +1 on the patch.
--
You received this message because you are listed in the owner
or CC fields of
Updates:
Labels: -NeedsBetterPatch NeedsReview
Comment #9 on issue 1467 by ondrej.certik: sorting values with imaginary
numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
(No comment was entered for this change.)
--
You received this message because you are listed
Comment #8 on issue 1467 by ted.ho...@earthlink.net: sorting values with
imaginary numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
Oops, here is the actual patch...
Attachments:
1467g.patch 2.9 KB
--
You received this message because you are listed in the own
Comment #7 on issue 1467 by ted.ho...@earthlink.net: sorting values with
imaginary numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
OK, here is a patch for Inequality and StrictInequality with some tests. I
think this takes care of the symptom,
but there are still som
Updates:
Status: Started
Labels: -NeedsReview NeedsBetterPatch
Comment #6 on issue 1467 by fab...@fseoane.net: sorting values with
imaginary numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
BTW, at some time we should rename all those __nonzero__ methods
Comment #5 on issue 1467 by fab...@fseoane.net: sorting values with
imaginary numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
Patch is nice and makes sense.
I don't like however that we are resorting to __nonzero__ when ordering
should be
done in __cmp__, but I could
Updates:
Labels: NeedsReview
Comment #4 on issue 1467 by ondrej.certik: sorting values with imaginary
numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
(No comment was entered for this change.)
--
You received this message because you are listed in the owner
or C
Comment #3 on issue 1467 by ted.ho...@earthlink.net: sorting values with
imaginary numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
Here is a small patch that seems to fix the immediate issue. I don't know
how these inequalities are supposed to
fit together, so this m
Updates:
Labels: Milestone-Release0.6.5
Comment #1 on issue 1467 by ondrej.certik: sorting values with imaginary
numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
Not sure at the moment, but this seems like a bug to me.
--
You received this message because you ar
Status: New
Owner:
Labels: Type-Defect Priority-Medium
New issue 1467 by smichr: sorting values with imaginary numbers in radical
http://code.google.com/p/sympy/issues/detail?id=1467
Sorting can be done with real and imaginary numbers, but if the imaginary
portion is in a radical, the sort
24 matches
Mail list logo