On Wed, Aug 12, 2009 at 8:00 AM, Vinzent
Steinberg<vinzent.steinb...@googlemail.com> wrote:
> 2009/8/12 Fabian Pedregosa <fab...@fseoane.net>
>>
>> Ondrej Certik wrote:
>> > This still fails in the latest branch:
>> >
>> > bin/test sympy/concrete/tests/test_sums_products.py -k
>> > test_composite_sums
>> >
>> > I just sent you a login info to my amd64 box offlist, please debug it.
>>
>>
>> This can be fixed by:
>>
>> diff --git a/sympy/core/assumptions.py b/sympy/core/assumptions.py
>> index e8a9fd6..8220550 100644
>> --- a/sympy/core/assumptions.py
>> +++ b/sympy/core/assumptions.py
>> @@ -101,7 +101,6 @@ class AssumeMeths(object):
>>          'complex        ->  commutative',
>>
>>          'odd            ==  integer & !even',
>> -        'even           ==  integer & !odd',
>>
>>          'real           ==  negative | zero | positive',
>>
>> sadly this makes other tests to fail.
>
> Any idea why this is necessary?
>
>> I propose pushing patches 1-4 (after fixing Vinzent's comments), work on
>> performance and then start removing the old system.
>
> +1
> As long as we don't drop the old system, we don't depend on functionality
> and performance of the new one.

Please prepare patch series, that can go in, I will look at it.

Ondrej

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"sympy-patches" group.
To post to this group, send email to sympy-patches@googlegroups.com
To unsubscribe from this group, send email to 
sympy-patches+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/sympy-patches?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to